Skip to content

feat: support openresty-1.25.3.1#10887

Merged
juststillthinking merged 33 commits into
apache:masterfrom
zll600:upgrade_openresty-1.25.3.1
Feb 26, 2024
Merged

feat: support openresty-1.25.3.1#10887
juststillthinking merged 33 commits into
apache:masterfrom
zll600:upgrade_openresty-1.25.3.1

Conversation

@zll600

@zll600 zll600 commented Jan 30, 2024

Copy link
Copy Markdown
Contributor

Description

Fixes #6403

Breaking Change:

  • If enable_http2 is set for any entry under apisix.ssl.listen or apisix.ssl.node_listen , all entries will enable http2. Becuase nginx-1.25.3 change the configuration of http2

from

server {
       listen 443 ssl http2;
}

to

server {
       http2 on;
       listen 443 ssl;
}

From user perspective, if I have following configuration, the port 9443 and 9444 will both support http2. The enable_http2: false takes no effect for port 9444

apisix:
  ssl:
    listen:
      - port: 9443
        enable_http2: true # enable http2
      - port: 9444
        enable_http2: false # disable http2.

doc for ngx.req.read_body from lua-nginx-module-0.10.26

Due to the stream processing feature of HTTP/2 or HTTP/3, this api could potentially block the entire request. Therefore, this api is effective only when HTTP/2 or HTTP/3 requests send content-length header. For requests with versions lower than HTTP/2, this api can still be used without any problems.

  • apisix will no longer support tongsuo. because QUIC of nginx needs QUIC TLS which tongsuo does not support currently.

related PRs:

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)

Comment thread t/stream-node/upstream-tls.t Outdated
@zll600 zll600 changed the title Upgrade openresty 1.25.3.1 chore: upgrade openresty 1.25.3.1 Feb 8, 2024
Comment thread apisix/cli/ngx_tpl.lua
Comment thread apisix/plugins/ext-plugin/init.lua
Comment thread apisix/core/request.lua
@zll600
zll600 marked this pull request as ready for review February 18, 2024 00:25
Comment thread apisix/cli/ops.lua
@juststillthinking

Copy link
Copy Markdown
Contributor

Please fix the code lint error in ci

@zll600

zll600 commented Feb 20, 2024

Copy link
Copy Markdown
Contributor Author

@monkeyDluffy6017 Could you help me rerun the CI?

Comment thread apisix/core/request.lua
Comment thread apisix/plugins/ext-plugin/init.lua
Comment thread ci/common.sh Outdated
Comment thread ci/common.sh Outdated
Comment thread utils/install-dependencies.sh Outdated
Comment thread t/plugin/proxy-rewrite2.t
Comment thread t/cli/test_main.sh Outdated
@juststillthinking juststillthinking changed the title chore: upgrade openresty 1.25.3.1 feat: support openresty-1.25.3.1 Feb 20, 2024
@juststillthinking

Copy link
Copy Markdown
Contributor

Maybe you could remove CI GM temporarily

Comment thread ci/centos7-ci.sh
Comment thread t/cli/test_main.sh
membphis
membphis previously approved these changes Feb 23, 2024

@membphis membphis left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

Comment thread t/plugin/azure-functions.t Outdated
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.

feat: apisix support QUIC+HTTP3

4 participants