search

TeslaGov / ngx-http-auth-jwt-module

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

Fix get_jwt() on ARM 32bit #111

Closed swaeberle closed 10 months ago

swaeberle commented 11 months ago

The sizeof() was operated on char * returning the architecture's pointer size, instead of the length of the char[]. On 64bit it worked only by chance, on 32bit the jwt_location was stripped at the wrong position.

JoshMcCullough commented 10 months ago

I believe we should be using srtlen(...) instead of sizeof(...)...thoughts?

JoshMcCullough commented 10 months ago

Closing this; fixing with #115.

JoshMcCullough commented 10 months ago

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

JoshMcCullough commented 10 months ago

Fixed in the 2.0.3 release. Thanks for pointing out the issue!