Skip to content

Conversation

@mjsalinger
Copy link
Contributor

…TokenProperties' argument

@mjsalinger mjsalinger changed the title Allow optional properties in token response by setting 'allowExtended… 252-Allow optional properties in token response by setting 'allowExtended… May 24, 2016
@mjsalinger
Copy link
Contributor Author

Fixes #252

maxtruxa added a commit to maxtruxa/node-oauth2-server that referenced this pull request May 27, 2016
… by setting 'allowExtendedTokenProperties' argument
@ruimarinho ruimarinho modified the milestones: 3.0.0, 3.0.0-b4 Oct 14, 2016
Copy link
Member

@maxtruxa maxtruxa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a minor style issue (a single superfluous space).

lib/server.js Outdated
options = _.assign({
accessTokenLifetime: 60 * 60, // 1 hour.
refreshTokenLifetime: 60 * 60 * 24 * 14 // 2 weeks.
refreshTokenLifetime: 60 * 60 * 24 * 14, // 2 weeks.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comments aren't aligned anymore.

@mjsalinger mjsalinger force-pushed the 252-Allow-optional-properties-in-token-response branch from 33d815b to b1ebe17 Compare October 25, 2016 10:45
@mjsalinger mjsalinger merged commit 9589be1 into oauthjs:master Oct 25, 2016
@mjsalinger mjsalinger deleted the 252-Allow-optional-properties-in-token-response branch October 25, 2016 10:46
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.

3 participants