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

Update auth.js #13

Closed
wants to merge 1 commit into from
Closed

Update auth.js #13

wants to merge 1 commit into from

Conversation

r2dev
Copy link

@r2dev r2dev commented Apr 26, 2017

use Bearer schema in authorization header

use Bearer schema in authorization header
@jamesbrewerdev
Copy link

Why is this change needed? Does using Token not work? Does Bearer give us something we haven't thought of yet?

@deksden
Copy link
Contributor

deksden commented May 4, 2017

Here is some discussion on this topic: gothinkster/realworld#81

I m agree that Bearer is a standard, but I suggest to ADD bearer support, not to replace Token, like in this PR.

My version of same topic: #27

@deksden deksden mentioned this pull request May 4, 2017
@apai4
Copy link
Member

apai4 commented May 4, 2017

Closed in favor of #27, thanks for the PR!

@apai4 apai4 closed this May 4, 2017
@r2dev r2dev deleted the patch-1 branch March 25, 2019 03:41
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants