Skip to content

chore: upgrade openresty version to v1.27.11#11936

Merged
shreemaan-abhishek merged 12 commits into
apache:masterfrom
shreemaan-abhishek:12711
Feb 10, 2025
Merged

chore: upgrade openresty version to v1.27.11#11936
shreemaan-abhishek merged 12 commits into
apache:masterfrom
shreemaan-abhishek:12711

Conversation

@shreemaan-abhishek

@shreemaan-abhishek shreemaan-abhishek commented Jan 22, 2025

Copy link
Copy Markdown
Contributor

Description

Upgrade openresty version to v1.27.1.1.

Checklist

  • I have explained the need for this PR and the problem it solves
  • I have explained the changes or the new features added to this PR
  • I have added tests corresponding to this change
  • I have updated the documentation to reflect this change
  • I have verified that this change is backward compatible (If not, please discuss on the APISIX mailing list first)

@shreemaan-abhishek
shreemaan-abhishek marked this pull request as ready for review February 6, 2025 15:21
@dosubot dosubot Bot added size:S This PR changes 10-29 lines, ignoring generated files. chore labels Feb 6, 2025
Comment thread t/node/grpc-proxy.t
--- error_code: 502
--- error_log
upstream: "grpc://127.0.0.1:80"
connect() failed (111: Connection refused) while connecting to upstream

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this change is needed as grpc proxy is done under subrequest, and the apisix custom logger (in apisix-nginx-module) overrides the original log handler of nginx and stores the reference to the original log handler in ctx.

but ngx_stash_ctx fails to store reference to that log handler. Thus the original log handler doesn't get executed and the upstream: "grpc: ..." field in error log is not present.

(matched_ssl.value.client.skip_mtls_uri_regex == nil)
local ok, err = ngx_ssl.verify_client(parsed_cert, depth,
-- TODO: support passing `trusted_certs` (3rd arg, keep it nil for now)
local ok, err = ngx_ssl.verify_client(parsed_cert, depth, nil,

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the verify client API has changed and it expects 4 parameters now, the 3rd parameter is trusted_certs, we can implement the support for trusted certs later.

@Revolyssup
Revolyssup self-requested a review February 6, 2025 15:32
@shreemaan-abhishek
shreemaan-abhishek merged commit 346722f into apache:master Feb 10, 2025
membphis added a commit that referenced this pull request May 12, 2026
… levels in v3.x sections (#13360)

- Replace `## Doc improvements` (3.13.0 / 3.14.0 / 3.14.1) and
  `## Developer productivity` (3.13.0) with H3 so they nest under the
  release header instead of starting a new top-level section.
- 3.12.0: correct OpenResty version typo `v1.27.11` -> `1.27.1.1` (#11936).
- 3.11.0: fix plugin name typo `attach-consmer-label` -> `attach-consumer-label` (#11604).
- 3.5.0: the `google-cloud-logging` `client_email` entry referenced #9813
  (proxy-rewrite); the actual PR is #9622.
- 3.0.0: fix `APSISIX` typo in the `enable_cpu_affinity` entry.

Refs #13359.
wistefan pushed a commit to wistefan/apisix that referenced this pull request Jun 16, 2026
… levels in v3.x sections (apache#13360)

- Replace `## Doc improvements` (3.13.0 / 3.14.0 / 3.14.1) and
  `## Developer productivity` (3.13.0) with H3 so they nest under the
  release header instead of starting a new top-level section.
- 3.12.0: correct OpenResty version typo `v1.27.11` -> `1.27.1.1` (apache#11936).
- 3.11.0: fix plugin name typo `attach-consmer-label` -> `attach-consumer-label` (apache#11604).
- 3.5.0: the `google-cloud-logging` `client_email` entry referenced apache#9813
  (proxy-rewrite); the actual PR is apache#9622.
- 3.0.0: fix `APSISIX` typo in the `enable_cpu_affinity` entry.

Refs apache#13359.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S This PR changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants