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

Allow custom https port #790

Merged
merged 1 commit into from Aug 25, 2014

Conversation

Projects
None yet
2 participants
@arjunadeltoso
Contributor

arjunadeltoso commented Aug 25, 2014

I need to run a vibe.d based web app on a cheap hosting shared server over ssl, currently setting the port to != 443 is not taken into consideration and methods like redirect will bring you back to 443.
I need the custom port to work since I can't bind to 443 not having root permission.

This change seems to be the only one needed (at least for my app).

@s-ludwig

This comment has been minimized.

Show comment
Hide comment
@s-ludwig

s-ludwig Aug 25, 2014

Member

Makes sense and looks like that even was the original intention of the code. Thanks, merging!

Member

s-ludwig commented Aug 25, 2014

Makes sense and looks like that even was the original intention of the code. Thanks, merging!

s-ludwig added a commit that referenced this pull request Aug 25, 2014

Merge pull request #790 from arjunadeltoso/https_custom_port
Fix fullURL to use the right https port.

@s-ludwig s-ludwig merged commit 52c699c into vibe-d:master Aug 25, 2014

1 check passed

continuous-integration/travis-ci The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment