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

serverUrl fails on non-standard port #79

Closed
Nicals opened this issue Dec 15, 2014 · 3 comments
Closed

serverUrl fails on non-standard port #79

Nicals opened this issue Dec 15, 2014 · 3 comments
Labels
bug it's broken! proxy hosting behind reverse proxies
Milestone

Comments

@Nicals
Copy link

Nicals commented Dec 15, 2014

On non standard port, the server url is

<protocol>://host(:<port>)?

Despite the example given, the actual way can't handle such requests. A request from http://localhost:8000 will return a server url as http://localhost:8000:8000 resulting on a browser error since the Location header is incorrect.

@nodiscc
Copy link
Member

nodiscc commented Dec 15, 2014

@Nicals Thanks, this is part of #17

@nodiscc nodiscc added the bug it's broken! label Dec 15, 2014
@nodiscc nodiscc added this to the 0.9beta milestone Jan 9, 2015
@nodiscc
Copy link
Member

nodiscc commented Jan 29, 2015

@Nicals Can you recheck agains the latest master? #98 has been merged and should fix this. Thanks

@nodiscc
Copy link
Member

nodiscc commented Feb 14, 2015

No reply, closing.

@nodiscc nodiscc closed this as completed Feb 14, 2015
@virtualtam virtualtam modified the milestones: 0.9.0, 0.5.0 Jul 30, 2015
@virtualtam virtualtam added the proxy hosting behind reverse proxies label Aug 23, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug it's broken! proxy hosting behind reverse proxies
Projects
None yet
Development

No branches or pull requests

3 participants