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

bug: The plain HTTP request was sent to HTTPS port [traffic-split] #9177

Closed
Ikalinichev opened this issue Mar 27, 2023 · 1 comment · Fixed by #9115
Closed

bug: The plain HTTP request was sent to HTTPS port [traffic-split] #9177

Ikalinichev opened this issue Mar 27, 2023 · 1 comment · Fixed by #9115
Assignees

Comments

@Ikalinichev
Copy link

Ikalinichev commented Mar 27, 2023

Description

Hi!
I have a problem combining two plugins: proxy-rewrite and traffic-split , in order to proxy requests to an https upstream. Below is the config. While I'am using only proxy-rewrite plugin ( traffic-split is deleted from the config) it works fine.
With traffic-split I get:

<title>400 The plain HTTP request was sent to HTTPS port</title> and as I can see in the log it sends request to "http://http-gateway.prodlike.dc.mts.ru" Does traffic-split work with https? What I am doing wrong?
{
    "uri": "/test1",
    "methods": ["POST"],
     "plugins": {
	"proxy-rewrite": {
            "_meta": {
                "filter": [["http_x-mts-msisdn","!","in",
                ["lemon","apple","bean"]]]},
            "uri": "/test2",
	    "scheme": "https"
        },
        "traffic-split": {
            "rules": [
                {"match": [{"vars": [["http_x-mts-msisdn","!","in",
                ["lemon","apple","bean"]]]}],
                    "weighted_upstreams": [
                        {
                            "upstream": {
                                "name": "upstream-A",
                                "scheme": "https",
				"pass_host": "node",
                                "nodes": {
                                    "http-gateway.test.com:443":1
                                }
                            },
                            "weight": 3
                        }
                    ]
                }
            ]
        }
    },
	"upstream": {
        "pass_host": "node",
        "scheme": "https",
        "nodes": {
            "http-gateway.test.com:443":1
        }
    }
}

Environment

  • APISIX version (run apisix version): 3.2.0
  • Operating system (run uname -a): Linux apisix-6d8d48b4c5-c2xxc 5.4.0-81-generic
  • OpenResty / Nginx version (run openresty -V or nginx -V):
    -nginx version: openresty/1.21.4.1
    built by gcc 9.3.1 20200408 (Red Hat 9.3.1-2) (GCC)
    built with OpenSSL 1.1.1s 1 Nov 2022
    TLS SNI support enabled
    configure arguments: --prefix=/usr/local/openresty/nginx --with-cc-opt='-O2 -DAPISIX_BASE_VER=1.21.4.1.7 -DNGX_GRPC_CLI_ENGINE_PATH=/usr/local/openresty/libgrpc_engine.so -DNGX_HTTP_GRPC_CLI_ENGINE_PATH=/usr/local/openresty/libgrpc_engine.so -DNGX_LUA_ABORT_AT_PANIC -I/usr/local/openresty/zlib/include -I/usr/local/openresty/pcre/include -I/usr/local/openresty/openssl111/include' --add-module=../ngx_devel_kit-0.3.1 --add-module=../echo-nginx-module-0.62 --add-module=../xss-nginx-module-0.06 --add-module=../ngx_coolkit-0.2 --add-module=../set-misc-nginx-module-0.33 --add-module=../form-input-nginx-module-0.12 --add-module=../encrypted-session-nginx-module-0.09 --add-module=../srcache-nginx-module-0.32 --add-module=../ngx_lua-0.10.21 --add-module=../ngx_lua_upstream-0.07 --add-module=../headers-more-nginx-module-0.33 --add-module=../array-var-nginx-module-0.05 --add-module=../memc-nginx-module-0.19 --add-module=../redis2-nginx-module-0.15 --add-module=../redis-nginx-module-0.3.9 --add-module=../ngx_stream_lua-0.0.11 --with-ld-opt='-Wl,-rpath,/usr/local/openresty/luajit/lib -Wl,-rpath,/usr/local/openresty/wasmtime-c-api/lib -L/usr/local/openresty/zlib/lib -L/usr/local/openresty/pcre/lib -L/usr/local/openresty/openssl111/lib -Wl,-rpath,/usr/local/openresty/zlib/lib:/usr/local/openresty/pcre/lib:/usr/local/openresty/openssl111/lib' --add-module=/tmp/tmp.xYfSxgSEeh/openresty-1.21.4.1/../mod_dubbo-1.0.2 --add-module=/tmp/tmp.xYfSxgSEeh/openresty-1.21.4.1/../ngx_multi_upstream_module-1.1.1 --add-module=/tmp/tmp.xYfSxgSEeh/openresty-1.21.4.1/../apisix-nginx-module-1.12.0 --add-module=/tmp/tmp.xYfSxgSEeh/openresty-1.21.4.1/../apisix-nginx-module-1.12.0/src/stream --add-module=/tmp/tmp.xYfSxgSEeh/openresty-1.21.4.1/../apisix-nginx-module-1.12.0/src/meta --add-module=/tmp/tmp.xYfSxgSEeh/openresty-1.21.4.1/../wasm-nginx-module-0.6.4 --add-module=/tmp/tmp.xYfSxgSEeh/openresty-1.21.4.1/../lua-var-nginx-module-v0.5.3 --add-module=/tmp/tmp.xYfSxgSEeh/openresty-1.21.4.1/../grpc-client-nginx-module-v0.4.2 --with-poll_module --with-pcre-jit --with-stream --with-stream_ssl_module --with-stream_ssl_preread_module --with-http_v2_module --without-mail_pop3_module --without-mail_imap_module --without-mail_smtp_module --with-http_stub_status_module --with-http_realip_module --with-http_addition_module --with-http_auth_request_module --with-http_secure_link_module --with-http_random_index_module --with-http_gzip_static_module --with-http_sub_module --with-http_dav_module --with-http_flv_module --with-http_mp4_module --with-http_gunzip_module --with-threads --with-compat --with-stream --with-http_ssl_module
  • etcd version, if relevant (run curl http://127.0.0.1:9090/v1/server_info):
  • APISIX Dashboard version, if relevant:
  • Plugin runner version, for issues related to plugin runners:
  • LuaRocks version, for installation issues (run luarocks --version):
@Ikalinichev Ikalinichev changed the title help request: The plain HTTP request was sent to HTTPS port help request: The plain HTTP request was sent to HTTPS port [traffic-split] Mar 28, 2023
@Ikalinichev
Copy link
Author

That works for me
#8996

@Ikalinichev Ikalinichev changed the title help request: The plain HTTP request was sent to HTTPS port [traffic-split] bug: The plain HTTP request was sent to HTTPS port [traffic-split] Mar 28, 2023
@shreemaan-abhishek shreemaan-abhishek self-assigned this Sep 5, 2023
@shreemaan-abhishek shreemaan-abhishek linked a pull request Sep 5, 2023 that will close this issue
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants