Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

README: JWT specs timestamps as seconds-since-epoch, but we use milliseconds #51

Merged
merged 1 commit into from Nov 20, 2013

Conversation

Projects
None yet
2 participants
Member

warner commented Aug 8, 2012

This documents our use of ms. Addresses id-specs#21.

@benadida: this needs updating in id-specs too. Which way do you want to go.. leave our implementation alone, or conform to upstream JWT?

@warner warner README: JWT specs timestamps as seconds-since-epoch, but we use milli…
…seconds

This documents our use of ms. Addresses PR #21.
93aa4dc
Contributor

lloyd commented on 93aa4dc Nov 20, 2013

r+

@lloyd lloyd added a commit that referenced this pull request Nov 20, 2013

@lloyd lloyd Merge pull request #51 from mozilla/warner/timestamp-specs
README: JWT specs timestamps as seconds-since-epoch, but we use milliseconds
558712b

@lloyd lloyd merged commit 558712b into master Nov 20, 2013

1 check passed

default The Travis build passed
Details

@callahad callahad deleted the warner/timestamp-specs branch Nov 20, 2013

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