-
Notifications
You must be signed in to change notification settings - Fork 304
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
Match IETF draft Authorization header #200
Comments
I think FCM doesn't support this yet, but still requires Thank you for opening the issue, I never would have noticed the change. |
Will follow up on this tomorrow, thanks for the bug. Does this currently break any browsers if we switch to this? |
I've just tried to run the tests after the change, they're failing with Chrome, working with Firefox. |
Schweet thank you for the info On Wed, 10 Aug 2016, 17:58 Marco, notifications@github.com wrote:
|
The
Authorization
header produced by this library includesBearer
as the scheme. The spec at https://tools.ietf.org/html/draft-ietf-webpush-vapid-01 states that the scheme should beWebPush
(https://tools.ietf.org/html/draft-ietf-webpush-vapid-01#section-3).The text was updated successfully, but these errors were encountered: