SkyLothar / lua-resty-jwt

JWT For The Great Openresty
Apache License 2.0
513 stars 179 forks source link

luarocks install lua-resty-jwt fails #62

Closed tsyrjanen closed 7 years ago

tsyrjanen commented 7 years ago

Luarocks installation fails.

In my ubuntu desktop machine:

$sudo luarocks install lua-resty-jwt

Warning: falling back to curl - install luasec to get native HTTPS support Installing https://luarocks.org/lua-resty-jwt-0.1.10-0.src.rock

Error: Couldn't extract archive .: unrecognized filename extension

And in my Dockerfile (based on CentOS):

Step 6/11 : RUN luarocks install lua-resty-jwt ---> Running in dd206fdf553c Warning: falling back to curl - install luasec to get native HTTPS support

Error: Unrecognized filename extension . Installing https://luarocks.org/lua-resty-jwt-0.1.10-0.src.rock... Using https://luarocks.org/lua-resty-jwt-0.1.10-0.src.rock... switching to 'build' mode The command '/bin/sh -c luarocks install lua-resty-jwt' returned a non-zero code: 1

patricksanders commented 7 years ago

It appears that the rockspec is broken: https://luarocks.org/manifests/skylothar/lua-resty-jwt-0.1.10-0.rockspec

The source URL is set to 'file://.', whereas 0.1.9 is a repo URL: https://luarocks.org/manifests/skylothar/lua-resty-jwt-0.1.9-0.rockspec

gimbel0893 commented 7 years ago

I've been using 0.1.10-0 for about 6 months with no problem. Even rebuilt it about a week ago with no problem.

Needed to rebuild today and now having the same issue as stated above.

The manifest was changed in last ~week? Can it be rolled back?

patricksanders commented 7 years ago

@gimbel0893 That seems unlikely, since 0.1.10-0 was released yesterday.

As a temporary workaround, I have pinned my version of lua-resty-jwt to >= 0.1.8, < 0.1.10.

gimbel0893 commented 7 years ago

Oops thanks for correction. I wasn't specifying version so started grabbing latest from yesterday then. Thanks much

Just tried 0.1.9 without issue

SkyLothar commented 7 years ago

😿 Sorry for the trouble. Just released 0.1.10-1 to fix this problem.

I'll add some tests for installation check after a new version is released.

tsyrjanen commented 7 years ago

Thanks, it works.

$sudo luarocks install lua-resty-jwt

Warning: falling back to curl - install luasec to get native HTTPS support Installing https://luarocks.org/lua-resty-jwt-0.1.10-1.src.rock lua-resty-jwt 0.1.10-1 is now installed in /usr/local (license: Apache License Version 2)

patricksanders commented 7 years ago

Thanks! Works for me as well.