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

OAuth 2 Authentication Scheme #6

Merged
merged 1 commit into from Dec 11, 2015

Conversation

Projects
None yet
3 participants
@kylef
Copy link
Member

kylef commented Oct 1, 2015

This pull request extends on the authentication framework (proposed in #4) to provide a base authentication scheme called "OAuth 2".

Dependencies:

  • #3: MSON Parameters and Headers
  • #4: Authentication Framework

@kylef kylef added the draft label Oct 1, 2015


+ Request
+ Headers
+ Authorization (Token)

This comment has been minimized.

@kylef

kylef Oct 1, 2015

Member

This unfortunately isn't correct but I'm unsure how to clearly express this. I'm wondering if anyone else has any elegant solutions.

Basically, the Bearer token will be in the Authorization header. It won't be in the header alone which is why this won't work. It will have a suffix like Authorization: Bearer f136803ab9c241079ba0cc1b5d02ee77.

## Rationale

You can use the OAuth 2 Authentication scheme by defining it within the
"Authentication Schemes" section of an API Blueprint.

This comment has been minimized.

@pksunkara

pksunkara Nov 3, 2015

Member

I consider OAuth 2 to be similar to Basic. A user should be able to use it directly in the Authenticated sections with some default values.

`Redirect Callback`.

The `Redirect` keyword has been added to represent the request that will be
made to the `Redirect Callback`.

This comment has been minimized.

@pksunkara

pksunkara Nov 3, 2015

Member

What? Using the name Redirect will be really confusing for people since it doesn't represent the response (which is a redirect) but instead represents the redirect callback.

@zdne zdne referenced this pull request Nov 9, 2015

Closed

Authentication syntax proposal #201

@zdne zdne referenced this pull request Dec 1, 2015

Open

Authentication #11

@zdne

This comment has been minimized.

Copy link
Contributor

zdne commented Dec 11, 2015

I am merging this to be shelved as a draft. The discussion to continue before going to the "accepted" stage.

zdne added a commit that referenced this pull request Dec 11, 2015

@zdne zdne merged commit c27fad0 into master Dec 11, 2015

@zdne zdne deleted the kylef/authentication-oauth2 branch Dec 11, 2015

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