-
-
Notifications
You must be signed in to change notification settings - Fork 690
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
Validate claims if configured and verify_signature is not. #608
Validate claims if configured and verify_signature is not. #608
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This definitely fixes my issue and my need too.
@jpadilla what do you think? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome, thanks! Could you add to the changelog as well?
I added a changelog entry, hope it's good enough. |
@jpadilla I'm sorry to bug you about this, would you be able to make a release for this fix? |
@jpadilla I'm sorry to bug you about this, would you be able to make a release for this fix? |
@jpadilla any idea of what I can do to help about this? |
Fixes #599