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

Do not fix lua-resty-jwt #363

Closed
wants to merge 2 commits into from
Closed

Conversation

kayano
Copy link

@kayano kayano commented Oct 16, 2020

fixes #362

@kayano kayano marked this pull request as ready for review October 16, 2020 11:52
@kayano
Copy link
Author

kayano commented Oct 16, 2020

@zandbelt is it possible to check travis on PR?

Copy link

@ledbruno ledbruno left a comment

Choose a reason for hiding this comment

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

we used this...seems ok

@kayano
Copy link
Author

kayano commented Nov 3, 2020

any chance to have it merged?

@kopernic-pl
Copy link

related/duplicate of #366 . But I'd want any of those two merged :)

@bodewig bodewig closed this in 68504de Nov 17, 2020
@bodewig
Copy link
Collaborator

bodewig commented Nov 17, 2020

I didn't want to pick up your changes to the Travis config, that's why I didn't merge the PR directly. Thanks you.

@bodewig
Copy link
Collaborator

bodewig commented Nov 17, 2020

which of course was, before I saw Travis needed to be changed as lua-resty-jwt uses a version 3 rockspec, sorry.

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.

Why lua-resty-jwt is fixed to version 0.2.0 in lua-resty-openidc 1.7.3 ?
4 participants