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

Is this project still active? #85

Open
agilezebra opened this issue Aug 22, 2018 · 10 comments
Open

Is this project still active? #85

agilezebra opened this issue Aug 22, 2018 · 10 comments

Comments

@agilezebra
Copy link

There are 3 PRs open for 6 months plus. We'd really like to use this project in production in our organisation but it difficult to do so comfortably if it's effectively abandoned.

@jgehrcke
Copy link

jgehrcke commented Aug 31, 2018

We use the project in production.

I have not followed the issues very recently. Do you think there is a currently known issue that is absolutely critical such as a security bug?

@jgehrcke
Copy link

There seems to be this fork: https://github.com/cdbattags/lua-resty-jwt/

@cdbattags what's the plan? :)

@jgehrcke
Copy link

I see: #79 (comment)

@cdbattags
Copy link

Hi folks! We’ve fully forked this and the fork is now even live on LuaRocks with the same name.

If you're using LuaRocks as a package manager then you're probably already using the fork in prod!

Otherwise, feel free to submit PRs on my branch and update to latest if you'd like to run latest version of OpenResty! Woo!

@jgehrcke
Copy link

@cdbattags so you have no plan trying to get your latest goodies into SkyLothar/lua-resty-jwt -- no?

What I would really love to see, be it here or in your repo, is a decent changelog :).

@cdbattags
Copy link

Nope, this maintainer is a-wall and until they come back and until we agree on a clear path forward all PRs should basically flow into my fork now!

As long as we as a group agree agree of course! The major part about a fork is getting it into the package managers (OPM and LuaRocks) but since this is done there's no reason to still attempt to get my changes into this repo.

Changelog easy to add for the latest version on mine but it's also very easy to sift through the tags.

The latest changes are really simple OpenSSL FFI stuffs to get OpenResty/NGINX 13+ working.

@cdbattags
Copy link

cdbattags commented Aug 31, 2018

I'll look through the latest PRs here this weekend and see if we can get them in mine but @agilezebra it would be great if you could open a PR to mine as well to preserve the commit authorship 😉

@zandbelt
Copy link
Contributor

As the author of lua-resty-openidc, a package that depends on lua-resty-jwt, I support this fork and IMHO it should be the new master because of the unhealthy state of the old repo.

@jgehrcke
Copy link

jgehrcke commented Sep 5, 2018

Thanks for all the feedback! That helps making decisions.

@jgehrcke
Copy link

jgehrcke commented Sep 5, 2018

One thought: there should be an attribution to the original author in the important places (in the README, maybe in the OPM/luarocks package descriptions).

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

4 participants