Yichun Zhang
Yichun Zhang
@haorenfsa This would break backward compatibility, right? Better avoid breaking it even though the current impl is wrong.
@chase Will you please review this PR? @haorenfsa found that you mistakenly use the MD5 digest of the user key instead of the user key itself in the `resty.aes` module.
@haorenfsa Yeah, seems like a github or travis bug.
@haorenfsa I'll look into this as soon as I can manage. Sorry for the delay on my side.
@rako9000 Thanks for looking into this. But the Travis CI testing is failing: https://travis-ci.org/openresty/lua-nginx-module/builds/139419383 Seems like your change leads to a lot of nginx warnings. That is, ``` nginx: [warn]...
@xiaolisheng They never meant to work with each other. But I agree it would be cool if they do. Will you contribute a pull request please?
@saga1015 Please, do not use Chinese here. This place is considered English only. If you really want to use Chinese, please join and post to the openresty (Chinese) mailing list...
It might worth implementing a dedicated API for this. It should be easy once we have the prepare/execute API.
@leetrout This is strange. Maybe you have invisible characters in your connect string? I've just tried the following minimal and standalone example on my side with the latest OpenResty: ```...
@leetrout Yeah, it's weird. Maybe some misconfigurations or other mistakes.