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.

Sign up for GitHub

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

Jump to bottom

"CryptoTokens" should match "JWT" spec #338

Closed
bshaffer opened this issue Feb 21, 2014 · 5 comments
Closed

"CryptoTokens" should match "JWT" spec #338

bshaffer opened this issue Feb 21, 2014 · 5 comments

Comments

@bshaffer
Copy link
Owner

CryptoToken is a word I made up. These are actually just JSON Web Tokens... why the distinction? Because I didn't know better. Let's fix it!
@bshaffer bshaffer changed the title Rename "CryptoTokens" to "JWT" "CryptoTokens" should match "JWT" spec Mar 21, 2014
@bshaffer
Copy link
Owner Author

There are other aspects of the JWT spec that should be following when this change is made:

  • "expires" => "exp"
  • "user_id" => "sub"
  • "client_id" => "iss"

@bshaffer bshaffer mentioned this issue Aug 12, 2014
Merged
@jaredk2g jaredk2g mentioned this issue Sep 16, 2014
Merged
bshaffer added a commit that referenced this issue Sep 16, 2014
@bshaffer
Merge pull request #460 from jaredtking/issue-338-rename-jwt-access-t…
82caca6 
…oken-claims

Rename JWT claims for JWT Access Tokens: #338
@bshaffer
Copy link
Owner Author

resolved in #460

@bbalet
Copy link

bbalet commented Jan 9, 2015

Please update online documentation :
http://bshaffer.github.io/oauth2-server-php-docs/overview/crypto-tokens/

@bshaffer
Copy link
Owner Author

bshaffer commented Jan 9, 2015

@bbalet I will as soon as the changes are merged into master for v1.6

@bshaffer
Copy link
Owner Author

@bbalet docs updated and release tagged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@bshaffer @bbalet