Add domain name as parameter... #317

Closed
opened 2026-01-19 18:29:52 +00:00 by michael · 3 comments
Owner

Originally created by @gufertum on GitHub.

I run the docker behind a proxy and the returned path still contains the localhost with the local port.
It would be nice to have a parameter to overrule this when starting the docker container...

Originally created by @gufertum on GitHub. I run the docker behind a proxy and the returned path still contains the localhost with the local port. It would be nice to have a parameter to overrule this when starting the docker container...
Author
Owner

@paolafrancesca commented on GitHub:

@gufertum I don't think that adding one or more parameters is the right solution

we should address the problem in server/handlers.go:467 using http header sent by the proxy.
I cannot test the current behaviour at the moment but will do in the next few days

@paolafrancesca commented on GitHub: @gufertum I don't think that adding one or more parameters is the right solution we should address the problem in server/handlers.go:467 using http header sent by the proxy. I cannot test the current behaviour at the moment but will do in the next few days
Author
Owner

@gufertum commented on GitHub:

Closed: setting up apache properly, fixed the issue (ProxyPreserveHost On)

@gufertum commented on GitHub: Closed: setting up apache properly, fixed the issue (ProxyPreserveHost On)
Author
Owner

@gufertum commented on GitHub:

I just created a pull request with the changes...
https://github.com/dutchcoders/transfer.sh/pull/141

@gufertum commented on GitHub: I just created a pull request with the changes... https://github.com/dutchcoders/transfer.sh/pull/141
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: dutchcoders/transfer.sh#317