mirror of
https://github.com/dutchcoders/transfer.sh.git
synced 2026-02-03 14:13:26 +00:00
Endless redirects using SSL/TLS with docker-image #141
Reference in New Issue
Block a user
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @yelodevopsi on GitHub.
Hi!
I'm trying to run the dutchcoders/transfer.sh:latest docker image, utilizing the TLS/SSL feature.
The closest I've come, using the host-os's cert and priv-keys:
But I'm faced with endless 308 redirects when doing this.
Is this because the Dockerfile only exposes 8080 I presume?
Apache proxy workaround
As a temporary workaround I'm able to make the frondend https using a Apache2 proxy setup, but download-links and all the URL's are still set to http://, thus most browser are blocking any downloads I'm sharing directly.
Suggestions
@paolafrancesca commented on GitHub:
@rompeldunk
there was actually a bug in the redirect handler, fixed on #441
@yelodevopsi commented on GitHub:
Seem that the error still persist after I purged and pulled :latest tag.
Could you confirm that you're able to run the SSL-based transfer.sh docker image?
I'm still testing with the command above.
@yelodevopsi commented on GitHub:
Thanks for the quick reply!
Yes, I corrected that error previously and tested without
truetoo - Still the redirect problem persists.The command below is tested without Apache2 (proxy) running, but gives the exact same output as if it was active (Virtucal config in my previous comment)
I've replaced my real domain with the my.domain.com and IP with 123.123.123.123 in the terminal code below:
Do you know or have any setups/examples with docker containers running transfer.sh with SSL?
I suspect there might be an issue with either any of the flags/a bug/ or that the dutchcoders/transfer.sh:latest image might not be applicable for using SSL?
Also: I tested with adding
EXPOSE 443to the Dockerfile and build it locally and test, but the redirect problem is the same.@paolafrancesca commented on GitHub:
@rompeldunk
--force-httpsis a boolean flag, you don't need to passtrueas a value, setting it is enoughpassing
trueseems to break the parsing of the rest of the command arguments and indeed your tls listener is never started.You can try the following:
It should work
@paolafrancesca commented on GitHub:
https://hub.docker.com/r/dutchcoders/transfer.sh/tags
it seems the docker image is not updated yet
@paolafrancesca commented on GitHub:
the workflow was still running on
master, fixed now, in a few minutes we should have the new docker image@yelodevopsi commented on GitHub:
Yes. Just confirmed it the latest docker pull! Thanks alot! 👍
@paolafrancesca commented on GitHub:
is it working now @rompeldunk ?