We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
There was an error while loading. Please reload this page.
validateScope
scope
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
Specify your setup
Describe the bug
Typings for validateScope don't correctly reflect that scope arg can be undefined
To Reproduce
Steps to reproduce the behavior:
model.validateScope()
Expected behavior
The typings correctly reflect that the scope argument is optional.
Screenshots
n/a
Additional context
The scope util that is responsible for parsing the scopes from a request (https://github.com/node-oauth/node-oauth2-server/blob/v5.0.0/lib/utils/scope-util.js#L10-L12) returns undefined when no scope parameter is present and this return value is then passed to the validateScope() method.
undefined
validateScope()
The text was updated successfully, but these errors were encountered:
dhensby
Successfully merging a pull request may close this issue.
Specify your setup
Describe the bug
Typings for
validateScope
don't correctly reflect thatscope
arg can be undefinedTo Reproduce
Steps to reproduce the behavior:
scope
parameter tomodel.validateScope()
method is `undefinedExpected behavior
The typings correctly reflect that the
scope
argument is optional.Screenshots
n/a
Additional context
The scope util that is responsible for parsing the scopes from a request (https://github.com/node-oauth/node-oauth2-server/blob/v5.0.0/lib/utils/scope-util.js#L10-L12) returns
undefined
when no scope parameter is present and this return value is then passed to thevalidateScope()
method.The text was updated successfully, but these errors were encountered: