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

check_token fails for endpoints without DOMAIN entry #16

Open
e-nouri opened this issue Jun 28, 2019 · 0 comments
Open

check_token fails for endpoints without DOMAIN entry #16

e-nouri opened this issue Jun 28, 2019 · 0 comments

Comments

@e-nouri
Copy link

e-nouri commented Jun 28, 2019

check_token fails for endpoints without DOMAIN entry, this is for example the case if you set SCHEMA_ENDPOINT, on the root /schema it will always fail because this resource_conf = config.DOMAIN[resource] it is a pretty easy fix, I changed it to this resource_conf = config.DOMAIN.get(resource, {}) .

I fixed it on my own fork, but did not add unit tests, if you accept pr, i will make one to fix this. I am leaving this for reference here, because it seems like the repo is not being updated lately.
Cheers,

pcvonz added a commit to pcvonz/eve-auth-jwt that referenced this issue Jun 9, 2020
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

1 participant