Skip to content
This repository was archived by the owner on Mar 6, 2026. It is now read-only.

feat: Improve handling of clock skew#858

Merged
arithmetic1728 merged 5 commits intogoogleapis:mainfrom
eesheesh:patch-1
Sep 7, 2021
Merged

feat: Improve handling of clock skew#858
arithmetic1728 merged 5 commits intogoogleapis:mainfrom
eesheesh:patch-1

Conversation

@eesheesh
Copy link
Copy Markdown
Contributor

@eesheesh eesheesh commented Sep 7, 2021

  1. Allow up to 60 seconds of clock skew.
  2. Make the error messages actionable to people who don't know the internals of OAuth/JWTs.

@eesheesh eesheesh requested review from a team, arithmetic1728 and silvolu as code owners September 7, 2021 20:26
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Sep 7, 2021
@eesheesh eesheesh changed the title Improve handling of clock skew feat: Improve handling of clock skew Sep 7, 2021
@tseaver tseaver added kokoro:force-run Add this label to force Kokoro to re-run the tests. owlbot:run Add this label to trigger the Owlbot post processor. labels Sep 7, 2021
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Sep 7, 2021
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 7, 2021
@arithmetic1728 arithmetic1728 added the owlbot:run Add this label to trigger the Owlbot post processor. label Sep 7, 2021
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Sep 7, 2021
@tseaver tseaver added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 7, 2021
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 7, 2021
@arithmetic1728 arithmetic1728 merged commit 45c4491 into googleapis:main Sep 7, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants