Skip to content
New issue

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

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

Already on GitHub? # to your account

Support Bearer for authentication scheme #81

Open
sandeesh opened this issue Apr 30, 2017 · 8 comments
Open

Support Bearer for authentication scheme #81

sandeesh opened this issue Apr 30, 2017 · 8 comments
Labels
Status: RFC awaiting for comments to be approved v2 changelog

Comments

@sandeesh
Copy link
Member

The RFC specification requires the use of 'Bearer' as the authentication scheme. The 'Bearer' scheme is also registered with the IANA. Most JWT packages and tools use 'Bearer' as the default scheme.

http://www.iana.org/assignments/http-authschemes/http-authschemes.xhtml
https://tools.ietf.org/html/rfc6750
https://tools.ietf.org/html/rfc7523
https://en.wikipedia.org/wiki/JSON_Web_Token

@Cameron-C-Chapman
Copy link
Member

I was thinking the same thing, Bearer is the standard, not Token.

@sandeesh sandeesh changed the title Changing authentication scheme to Bearer Suggestion : Changing authentication scheme to Bearer Apr 30, 2017
@EricSimons EricSimons changed the title Suggestion : Changing authentication scheme to Bearer Support Bearer for authentication scheme Apr 30, 2017
@EricSimons
Copy link
Member

Just talked with @apai4 and we're thinking of having the backends support both so that there's backwards compatibility. It's not uncommon to see non-Bearer versions in libs, so doing a hard swap for all FE's/BE's seems unnecessary, but eventually we may just deprecate token/etc.

@Cameron-C-Chapman
Copy link
Member

Supporting both works. I've seen Token, Auth, Authorization, and Bearer but Bearer is the standard. I'll get the Spring backend setup to support either and I think that's a good solution going forward.

@jamesbrewerdev
Copy link

Is there anything left to add to this conversation or do we have a consensus? If the latter, should this issue be closed?

@sandeesh
Copy link
Member Author

sandeesh commented May 5, 2017

So the conclusion is that we enable support for both the schemes across all backends then?

@Cameron-C-Chapman
Copy link
Member

I think so @SandeeshS but we should probably add some notes to the spec before we close this out.

@sandeesh
Copy link
Member Author

sandeesh commented May 5, 2017

@Cameron-C-Chapman absolutely. There's almost 25+ backends in progress/complete at the moment. We need to make sure we inform changes such as these to all the current backend developers in a proper manner. Also document this in the both frontend and backend spec.

@jamesbrewerdev
Copy link

Sounds good. I'll leave yall to it then.

Anytime we're talking about adding something to the spec, it's necessary to wait until @EricSimons signs off on it. He's got the best view of the project as a whole and is best equipped to make sure a call and bring our attention to things we haven't considered.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Status: RFC awaiting for comments to be approved v2 changelog
Projects
None yet
Development

No branches or pull requests

5 participants