Skip to content
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

[DNM] #5

Open
wants to merge 114 commits into
base: master
Choose a base branch
from
Open

[DNM] #5

wants to merge 114 commits into from

Conversation

sambacha
Copy link
Owner

No description provided.

dhritzkiv and others added 30 commits August 14, 2017 16:43
Fix link to RFC6750 standard
related to a NodeJS (nodejs/node#7074) and furthermore
V8 bug (https://bugs.chromium.org/p/v8/issues/detail?id=3637); replaced
seconds calculation with milliseconds.
readme: Update Slack badge and link
set numArgs for promisify of generateAuthorizationCode
fix: issue correct expiry dates for tokens
According to https://tools.ietf.org/html/rfc6749#section-4.1.2.1
once the redirect_uri & client_id is correct authorization server should
inform the clinet, that user denied access.

The change is to move validation of resource owner approval after the
redirect_uri & client identifier validation so the correct redirect url
is computed
Fix `hasOwnProperty is not a function` error
…horize

 fix: validate requested scope on authorize request
…-request

Extend model object with request context
Rectified the link to RFC 6750 paper
Correct `generateAccessToken` invocation to include the correct parameters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.