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

[Feature] clock skew #492

Merged
merged 2 commits into from
Jun 1, 2018
Merged

[Feature] clock skew #492

merged 2 commits into from
Jun 1, 2018

Conversation

patrickjDE
Copy link
Contributor

  • implement clock_skew parameter
  • update unit test

This PR introduces the clock skew parameter, that allows to account for time differences between servers.

* implement clock_skew parameter
* update unit test
@chalasr
Copy link
Collaborator

chalasr commented May 16, 2018

Could you also update the LcobucciJWSProvider class? I can do it otherwise.
Thanks for proposing this feature

@patrickjDE
Copy link
Contributor Author

I needed this for a project I'm working on, so why not share the implementation 😄
I'll update the other provider class in the next days.

@patrickjDE
Copy link
Contributor Author

Finally had the time for it, also updated the corresponding tests.

@chalasr
Copy link
Collaborator

chalasr commented Jun 1, 2018

Thanks @patrickjDE for working on this feature, this is much appreciated.

@chalasr chalasr merged commit c0e919a into lexik:master Jun 1, 2018
chalasr added a commit that referenced this pull request Jun 1, 2018
This PR was merged into the 2.x-dev branch.

Discussion
----------

[Feature] clock skew

* implement clock_skew parameter
* update unit test

This PR introduces the clock skew parameter, that allows to account for time differences between servers.

Commits
-------

c0e919a implement clock skew for LcobucciJWSProvider, update unit tests
3efe31d [Feature] clock skew
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants