This is an automated email from the ASF dual-hosted git repository. starsz pushed a commit to branch chore_add_http2_testcase in repository https://gitbox.apache.org/repos/asf/apisix.git
commit b2afa089241af4752d37387a0aa8a25f221e183f Author: Peter Zhu <[email protected]> AuthorDate: Fri Oct 13 17:13:23 2023 +0800 fix --- .github/workflows/build.yml | 9 ++++----- ci/common.sh | 2 +- t/plugin/limit-conn3.t | 18 ++---------------- 3 files changed, 7 insertions(+), 22 deletions(-) diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml index 95e72fe57..4971186a7 100644 --- a/.github/workflows/build.yml +++ b/.github/workflows/build.yml @@ -46,11 +46,6 @@ jobs: with: submodules: recursive - - name: Setup Go - uses: actions/setup-go@v4 - with: - go-version: "1.17" - - name: Cache deps uses: actions/cache@v3 env: @@ -171,6 +166,10 @@ jobs: TEST_FILE_SUB_DIR: ${{ matrix.test_dir }} run: sudo -E ./ci/${{ matrix.os_name }}_runner.sh script + - name: Setup tmate session + if: ${{ failure() }} + uses: mxschmitt/action-tmate@v3 + - if: ${{ steps.cache-images.outputs.cache-hit != 'true' }} name: Save docker images run: | diff --git a/ci/common.sh b/ci/common.sh index 3ad1e651c..a2a46a7d4 100644 --- a/ci/common.sh +++ b/ci/common.sh @@ -149,5 +149,5 @@ GRPC_SERVER_EXAMPLE_VER=20210819 linux_get_dependencies () { apt update - apt install -y cpanminus build-essential libncurses5-dev libreadline-dev libssl-dev perl libpcre3 libpcre3-dev libldap2-dev + apt install -y cpanminus build-essential libncurses5-dev libreadline-dev libssl-dev perl libpcre3 libpcre3-dev libldap2-dev curl } diff --git a/t/plugin/limit-conn3.t b/t/plugin/limit-conn3.t index 9f698c329..e778962fe 100644 --- a/t/plugin/limit-conn3.t +++ b/t/plugin/limit-conn3.t @@ -37,11 +37,6 @@ add_block_preprocessor(sub { my $port = $ENV{TEST_NGINX_SERVER_PORT}; my $TEST_NGINX_HTML_DIR ||= html_dir(); - my $config = $block->config // <<_EOC_; - listen unix:$TEST_NGINX_HTML_DIR/nginx.sock ssl; -_EOC_ - - $block->set_value("config", $config); if (!$block->request) { $block->set_value("request", "GET /t"); @@ -55,6 +50,7 @@ _EOC_ run_tests; __DATA__ + === TEST 1: create route with limit-conn plugin --- config location /t { @@ -96,9 +92,6 @@ passed === TEST 2: create ssl(sni: www.test.com) ---- yaml_config -apisix: - node_listen: 1984 --- config location /t { content_by_lua_block { @@ -130,14 +123,7 @@ passed === TEST 3: use HTTP version 2 to request ---- yaml_config -apisix: - ssl: - enable: true - listen: # APISIX listening port for HTTPS traffic. - - port: 9443 - enable_http2: true --- exec -curl --http2 --parallel -k https://www.test.com:9443/limit_conn https://www.test.com:9443/limit_conn --resolve www.test.com:9443:127.0.0.1 +curl --http2 --parallel -k https://www.test.com:1994/limit_conn https://www.test.com:1994/limit_conn --resolve www.test.com:1994:127.0.0.1 --- response_body_like 503 Service Temporarily Unavailable.*.hello world
