Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Safari 11.1 shouldn't polyfill fetch #1252

Open
Morozzzko opened this issue Jun 21, 2022 · 3 comments
Open

Safari 11.1 shouldn't polyfill fetch #1252

Morozzzko opened this issue Jun 21, 2022 · 3 comments
Labels
library Relates to an Origami library service Relates to an Origami service

Comments

@Morozzzko
Copy link

What

Safari 11.1 shouldn't receive polyfill for fetch because it already has it

Details

We can see that https://caniuse.com/?search=fetch says that Safari supports fetch since 10.1

safari

I've checked it on Browserstack (Mac OS Monterey, Safari 11.1), and it indeed loads a polyfill:

v3

even though the browser has it anyway

browser

@github-actions github-actions bot added the service Relates to an Origami service label Jun 21, 2022
@romainmenke
Copy link
Collaborator

The fetch polyfill also includes patches for Request, Response and Headers.
Some of these are required up to Safari 12.1

mdn/browser-compat-data#2006 (comment)

@ffxsam
Copy link

ffxsam commented Aug 31, 2022

This might be related, but I noticed IntersectionObserver being polyfilled in Safari on iOS 15, when it already exists. So there might be some feature detection issues in Safari in general.

@romainmenke
Copy link
Collaborator

@ffxsam That is because of this issue and is unrelated to fetch : #1192

@JakeChampion JakeChampion transferred this issue from polyfillpolyfill/polyfill-service Jan 9, 2023
@github-actions github-actions bot added the library Relates to an Origami library label Jan 9, 2023
@robertboulton robertboulton removed this from Backlog in Origami ✨ Jul 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
library Relates to an Origami library service Relates to an Origami service
Projects
None yet
Development

No branches or pull requests

3 participants