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

Mercure send 401 error when JWT token contain a exp field #522

Closed
GRUXXKi opened this issue May 21, 2021 · 5 comments
Closed

Mercure send 401 error when JWT token contain a exp field #522

GRUXXKi opened this issue May 21, 2021 · 5 comments

Comments

@GRUXXKi
Copy link

GRUXXKi commented May 21, 2021

I've opened an issue here symfony/mercure-bundle#56 on the Symfony Mercure bundle also.

It appears that when the JWT token contain field like "exp" in payload , it returns a 401 error.

As it could be an interesting value, maybe it should be accepted ?

mercure-bundle 0.3 and mercure server 0.11.3.

@norkunas
Copy link

Possibly related to #404 ?

@GRUXXKi
Copy link
Author

GRUXXKi commented Jun 15, 2021

Not sure. Is'nt it fixed now the #404 ?

@norkunas
Copy link

Sorry, missed your mercure version :) Not sure then, for us everything works on v0.10 with simple exp as integer.

@dunglas
Copy link
Owner

dunglas commented Jun 24, 2021

This has indeed been fixed by #404.
By the way, we're in the process of migrating to golang-jwt/jwt instead of using our own fork (which is necessary just because of this issue). We're just waiting for golang-jwt/jwt#15 to be merged.

Feel free to reopen if you still have the issue with the last version.

@dunglas dunglas closed this as completed Jun 24, 2021
@GRUXXKi
Copy link
Author

GRUXXKi commented Jun 25, 2021

Yes it seems alright now !

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

No branches or pull requests

3 participants