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.

Sign up for GitHub

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

Jump to bottom

Non GET HTTP to HTTPS redirect does not work properly. #1738

Open
OliverRhyme opened this issue Aug 18, 2021 · 1 comment · May be fixed by #1737
Open

Non GET HTTP to HTTPS redirect does not work properly. #1738

OliverRhyme opened this issue Aug 18, 2021 · 1 comment · May be fixed by #1737
Labels
kind/bug Issue reporting a bug

Comments

@OliverRhyme
Copy link

Calling post request to http endpoint will instead call get method on the reverse proxy.

See logs

nginx-proxy          | nginx.1     | <REDACTED> - - [18/Aug/2021:09:23:31  0000] "POST /api/v1/register HTTP/1.1" 301 169 "-" "PostmanRuntime/7.28.2" "-"
asi-server           | 2021-08-18 09:23:32.081 [DefaultDispatcher-worker-1] TRACE ktor.application - 404 Not Found: GET - /register
nginx-proxy          | nginx.1     | <REDACTED> - - [18/Aug/2021:09:23:32  0000] "GET /api/v1/register HTTP/1.1" 404 0 "http://<REDACTED>/api/v1/register" "PostmanRuntime/7.28.2" "172.19.0.5:8080"

But calling https directly works just fine

asi-server           | 2021-08-18 09:27:17.602 [DefaultDispatcher-worker-3] TRACE ktor.application - 400 Bad Request: POST - /register
nginx-proxy          | nginx.1     | <REDACTED> - - [18/Aug/2021:09:27:17  0000] "POST /api/v1/register HTTP/1.1" 400 62 "-" "PostmanRuntime/7.28.2" "<REDACTED>"

My docker-compose env variables is

- VIRTUAL_HOST=REDACTED
- VIRTUAL_PORT=8080
- VIRTUAL_PATH=/api/v1/
- VIRTUAL_DEST=/

Originally posted by @OliverRhyme in #1733 (comment)
@OliverRhyme
Copy link
Author

Test fix: #1737

@buchdag buchdag changed the title HTTP to HTTPS redirect does not work properly. Non GET HTTP to HTTPS redirect does not work properly. Aug 18, 2021
@OliverRhyme OliverRhyme linked a pull request Aug 19, 2021 that will close this issue
Open
@buchdag buchdag added the kind/bug Issue reporting a bug label Jan 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
kind/bug Issue reporting a bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: redirect non-GET methods using 308 instead of 301 junderw/nginx-proxy
2 participants
@buchdag @OliverRhyme