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

request help: Can the port be set for http_to_https in the redirect plugin? #4400

Closed
ieayoio opened this issue Jun 9, 2021 · 5 comments · Fixed by #6686
Closed

request help: Can the port be set for http_to_https in the redirect plugin? #4400

ieayoio opened this issue Jun 9, 2021 · 5 comments · Fixed by #6686
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@ieayoio
Copy link

ieayoio commented Jun 9, 2021

Issue description

Can the port be set for http_to_https in the redirect plugin?

The default https port of apisix is 9443, but after I set http_to_https to true,
visiting "http://test.com:9080/service/"
will redirect to "https://test.com/service/",
my expected to jump to "https://test.com:9443/service/"

Environment

  • apisix version (cmd: apisix version):2.6
  • OS (cmd: uname -a):Linux xxxxx 4.19.90-2012.5.0.0054.oe1.x86_64 #1 SMP Tue Dec 22 15:58:47 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
  • OpenResty / Nginx version (cmd: nginx -V or openresty -V):
nginx version: openresty/1.19.3.1
built by gcc 8.3.1 20190311 (Red Hat 8.3.1-3) (GCC)
built with OpenSSL 1.1.1h  22 Sep 2020 (running with OpenSSL 1.1.1k  25 Mar 2021)
  • etcd version, if have (cmd: run curl http://127.0.0.1:9090/v1/server_info to get the info from server-info API):
{"hostname":"xxxxxx","boot_time":1623142227,"etcd_version":"3.4.0","up_time":93656,"last_report_time":1623235829,"id":"xxxxx","version":"2.6"}
  • apisix-dashboard version, if have:
  • luarocks version, if the issue is about installation (cmd: luarocks --version):/usr/local/bin/luarocks 3.4.0
@tokers
Copy link
Contributor

tokers commented Jun 9, 2021

@ieayoio Currently we can't, but PR's welcome to extend it.

@ieayoio ieayoio closed this as completed Jun 10, 2021
@rockjing
Copy link

I have the same issue. So what's update ?

@membphis membphis reopened this Mar 13, 2022
@membphis
Copy link
Member

I reopened this issue, welcome PR from anyone

@tokers
Copy link
Contributor

tokers commented Mar 14, 2022

I have the same issue. So what's update ?

Would you like to try it?

@juzhiyuan juzhiyuan added the enhancement New feature or request label Mar 14, 2022
@moonming moonming added the good first issue Good for newcomers label Mar 16, 2022
@kwanhur
Copy link
Contributor

kwanhur commented Mar 22, 2022

I'll try to solve it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants