search

BTBurke / caddy-jwt

JWT middleware for the Caddy server
MIT License
113 stars 40 forks source link

Update for caddy repo move #52

Closed hairyhenderson closed 5 years ago

hairyhenderson commented 5 years ago

Fixes #51

Caddy has moved from github.com/mholt/caddy to github.com/caddyserver/caddy, and so some modules no longer build properly as of Caddy v1.0.1

This updates caddy-jwt so it'll work!

I saw @mholt file a few issues in other module repos, but I guess he missed this one 😉 - see also the note in https://github.com/caddyserver/caddy/releases/tag/v1.0.1

Signed-off-by: Dave Henderson dhenderson@gmail.com

mholt commented 5 years ago

I did file an issue for this one :wink: https://github.com/BTBurke/caddy-jwt/issues/51

But not a pull request, which is even better. Thanks for this!!

hairyhenderson commented 5 years ago

@mholt not sure how I missed the issue - sorry! 😂

xombiemp commented 5 years ago

Can you please merge this? I'm getting an error trying to install using this Archlinux package and I think this is why. https://aur.archlinux.org/packages/caddy/

hairyhenderson commented 5 years ago

@BTBurke can you have a look at this please? 🙂

BTBurke commented 5 years ago

I don’t have time right now but I’ll get to it as soon as I can On Jul 10, 2019, 14:27 -0400, Dave Henderson notifications@github.com, wrote:

@BTBurke can you have a look at this please? 🙂 — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

BTBurke commented 5 years ago

Merged. Thanks for doing that.

hairyhenderson commented 5 years ago

Thanks @BTBurke, I (and I'm sure many others) appreciate it! 🙂