-
Notifications
You must be signed in to change notification settings - Fork 104
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
chore: added worker support #163
Conversation
Ah cool, in the meantime, Inrupt shipped their own fetch library which uses native feature if available: https://npm.im/@inrupt/universal-fetch Which doesn't do polyfilling for browsers, given the wide support for fetch these days |
@lquixada 👏 THANK YOU ! |
@spencercap it'll be on version 4. you can test it using |
@lquixada it works! thanks so much. the only thing i needed to change in this library's fork to get it working was to remove the it seems to me that the CF worker's |
@spencercap I'm glad that it worked out for you! Please note that |
This work enables cross-fetch to work in workers environment such as Service Workers and Cloudflare Workers.
This PR addresses issues #69, #78 and #142.