search

cdbattags / lua-resty-jwt

JWT For The Great Openresty
Apache License 2.0
146 stars 44 forks source link

fix OpenSSL 1.1 support #4

Closed mikz closed 6 years ago

mikz commented 6 years ago

Fixed allocating HMAC objects instead of EVP MD CTX.

bad argument #1 to 'EVP_DigestInit_ex' (cannot convert 'struct hmac_ctx_st ' to 'struct env_md_ctx_st ')

@cybrq-as this looks like a copy-paste error from the hmac.lua.

ghost commented 6 years ago

Good catch.

mikz commented 6 years ago

@cdbattags this is quite critical as none of the EVP functionality works on OpenSSL 1.1. Could you merge this and cut another release?

cdbattags commented 6 years ago

Aha! You caught it too. I have the same locally but I'll use this PR instead and toss ya the credit. I think I'm gonna go ahead and version bump to v0.2.0 as well and fix/remove the corresponding releases for OPM and LuaRocks.

Edit:

Please refer to my changes at https://github.com/cdbattags/lua-resty-jwt/pull/5! I'll cut this as v0.2.0!