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

Nens auth client #13

Merged
merged 6 commits into from
Mar 29, 2024
Merged

Nens auth client #13

merged 6 commits into from
Mar 29, 2024

Conversation

JJFlorian
Copy link
Contributor

Hoi Reinout,

Ik heb een poging gedaan om de nens-auth-client te installeren. Leek me interessant om zelf eens te proberen. Ik denk dat ik aardig ver ben gekomen, maar ik loop momenteel tegen deze fout aan::

invalid_token: The token is not valid as it was issued in the future

Ik hoop dat dit ligt aan bepaalde configuratie waar ik niet bij kan. Als het ergens anders aan ligt, dan hoor ik het graag

@JJFlorian JJFlorian requested a review from reinout March 28, 2024 15:14
@JJFlorian
Copy link
Contributor Author

Ik weet niet of jij iets had gedaan al Reinout, maar vandaag werkt de client wel.

@reinout
Copy link
Member

reinout commented Mar 29, 2024

Ik heb niks gedaan :-) Ik zal nu even naar je PR kijken.

Dit soort "het komt uit de toekomst" errors kan gebeuren als de interne klok van je docker niet klopt met het OS waarin je browser draait,

@reinout
Copy link
Member

reinout commented Mar 29, 2024

Die JWT tokens die op de achtergrond worden gebruikt hebben allemaal een ingebouwde hard afgedwongen timestamp met beperkte geldigheid (minuutje of twee ofzo). Als windows en docker dan hun tijd niet netjes coördineren kan de docker een timestamp gebruiken die bij windows niet meer klopt.

Dat dit alles zo nauwkeurig kan werken is met dank aan "Father Time" (https://en.wikipedia.org/wiki/David_L._Mills) die afgelopen januari is overleden :-)

Copy link
Member

@reinout reinout left a comment

Choose a reason for hiding this comment

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

Ziet er gewoon goed uit :-)

@JJFlorian JJFlorian merged commit 0a9a43b into main Mar 29, 2024
1 check passed
@JJFlorian JJFlorian deleted the nens_auth_client branch March 29, 2024 10:00
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.

2 participants