Skip to content
This repository has been archived by the owner on Mar 21, 2024. It is now read-only.

Consider using FetchEvent to append access token to requests #11

Open
manifest opened this issue Jan 19, 2019 · 3 comments
Open

Consider using FetchEvent to append access token to requests #11

manifest opened this issue Jan 19, 2019 · 3 comments

Comments

@manifest
Copy link
Member

manifest commented Jan 19, 2019

FetchEvent is now shipped by all major browser vendors
https://jakearchibald.github.io/isserviceworkerready/#fetch-event
https://w3c.github.io/ServiceWorker/#fetchevent

It may be used to add "authorization" HTTP header with access token of the current user for specified list of URIs.

@rogaldh
Copy link
Contributor

rogaldh commented Jan 21, 2019

Still not sure we have to implement this. There is a lack of support by Safari IOS. Edge (<17) does not play well with FetchEvent also.

@manifest
Copy link
Member Author

manifest commented Jan 21, 2019

I don't think this library should target mobile because there are other ways to authenticate users on mobile. Edge (<17) means two latest versions already support FetchEvent.

@rogaldh
Copy link
Contributor

rogaldh commented Jan 21, 2019

We aimed to use that library at applications which run on mobile and desktop.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Development

No branches or pull requests

2 participants