JWTRefreshTokenBundle icon indicating copy to clipboard operation
JWTRefreshTokenBundle copied to clipboard

Published 20 hours ago • verified publisher icon markitosgv

Fix token expiration date

Open ricardodevries opened this issue 3 years ago • 1 comments

Hello maintainers,

Instead of using time() + $this->ttl my suggestion is to use the getValid() method of the token. If you request a new token through /api/token/refresh, you get the latest unix time while the token may expire another timestamp. This fix uses the getValid method and turns it into an timestamp.

As always I am open to suggestions, critical feedback leading to improvements on this PR or comments 👍

ricardodevries avatar May 31 '22 11:05 ricardodevries

I'm checking up to see if anything else is needed to finish and merge this PR.

ricardodevries avatar Aug 16 '22 05:08 ricardodevries

Just following up. I apologize in advance for bothering you.

ricardodevries avatar Oct 16 '22 08:10 ricardodevries