Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

OpenResty 1.13.6.2 #733

Merged
merged 5 commits into from
Jun 11, 2018
Merged

OpenResty 1.13.6.2 #733

merged 5 commits into from
Jun 11, 2018

Conversation

mikz
Copy link
Contributor

@mikz mikz commented May 30, 2018

Upgrade OpenResty to 1.13.6.2. See the changelog: http://openresty.org/en/changelog-1013006.html
Major change is OpenSSL 1.1, so some FFI code needed to be changed: (cdbattags/lua-resty-jwt#4)

using 3scale/s2i-openresty#58

@eguzki
Copy link
Member

eguzki commented May 30, 2018

"Starting from openresty 1.13.6.2, all our official pre-built packages have switched from OpenSSL 1.0.2 over to the OpenSSL 1.1.0 series"

openresty/openresty#372 (comment)

@mikz mikz changed the title [ci] try latest openresty image [ci] try latest openresty image (1.13.6.2) May 30, 2018
@mikz
Copy link
Contributor Author

mikz commented May 30, 2018

@eguzki if this breaks your local setup we could vendor that resty.evp (SkyLothar/lua-resty-jwt#79) change into APIcast until it is merged.

@eguzki
Copy link
Member

eguzki commented May 30, 2018

Just wanted bring heads up on this issue.

I can manage to have local setup up and running.

@mikz mikz force-pushed the openresty-1.13.6.2 branch from 79fc056 to 0add88a Compare June 4, 2018 12:02
@mikz mikz requested a review from a team as a code owner June 7, 2018 07:34
@mikz mikz force-pushed the openresty-1.13.6.2 branch from 96c626f to ec1b5f3 Compare June 11, 2018 06:45
@mikz mikz requested a review from a team as a code owner June 11, 2018 06:45
mikz added 4 commits June 11, 2018 08:50
Setting cjson.array_mt would not serialize non integer elements
resulting in always empty array response.
@mikz mikz force-pushed the openresty-1.13.6.2 branch from ec1b5f3 to b2d1769 Compare June 11, 2018 06:51
@mikz mikz changed the title [ci] try latest openresty image (1.13.6.2) OpenResty 1.13.6.2 Jun 11, 2018
@davidor davidor merged commit 8244ef1 into master Jun 11, 2018
@davidor davidor deleted the openresty-1.13.6.2 branch June 11, 2018 08:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants