Skip to content

Conversation

@aldas
Copy link
Contributor

@aldas aldas commented Jun 10, 2021

Add JWTConfig.ParseTokenFunc info and warning. This is documentation for labstack/echo#1887

@aldas aldas force-pushed the jwt_parsetokenfunc branch from 581f269 to 7ecf285 Compare June 10, 2021 21:27
Copy link
Contributor

@pafuent pafuent left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Contributor

@lammel lammel left a comment

Choose a reason for hiding this comment

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

Please apply some of the wording improvements as you see fit.
LGTM overall!

@aldas aldas requested a review from lammel June 13, 2021 17:57
Copy link
Contributor

@lammel lammel left a comment

Choose a reason for hiding this comment

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

Nice!

@lammel lammel merged commit b27110a into labstack:master Jun 13, 2021
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.

3 participants