search

TeslaGov / ngx-http-auth-jwt-module

Secure your NGINX locations with JWT
MIT License
317 stars 122 forks source link

jwt_free #35

Closed TimUnderhay closed 6 years ago

TimUnderhay commented 6 years ago

This PR is my stab at resolving #33. It runs jwt_free either in the 'goto redirect' block or at the end of the normal ngx_http_auth_jwt_handler path. It initialises *jwt with a null value so that the redirect block can detect whether the normal path has gotten to init jwt yet.

fitzyjoe commented 6 years ago

Looks good! It won't let me merge because it says that there are conflicts. I think that's because 4 days ago I removed the ".so" file and your commit contains updates to that file. Can you remove that file from this branch and maybe it'll let me merge. Thx.

TimUnderhay commented 6 years ago

Sure thing - done.

fitzyjoe commented 6 years ago

It still won't let me merge... so I created a new branch and made another pull request and merged it. Thanks.

https://github.com/TeslaGov/ngx-http-auth-jwt-module/pull/36