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

fix reverse-proxy testnet foo #8

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

BugRoger
Copy link

@BugRoger BugRoger commented Jul 8, 2017

I'm hosting my wallet behind a reverse-proxy server. It adds SSL termination and exposes the apiserver on port 443. In the browser it is called like https://mywallet.com.

Now that there is no port in the URL the UI connects against the Testnet port, due an unfortunate check that fails if no port is given.

This patch fixes this problem and makes the intention clear, that only with the special testnet port the UI connects to Testnet.

@Doncode
Copy link

Doncode commented Jul 30, 2017

Nice fix. I did the same thing.
Please merge it

if ('port' in window.location && window.location.port != "6876") {
	$(".testnet_only").hide();
} else {
	NRS.isTestNet = true;
	$(".testnet_only, #testnet_login, #testnet_warning").show();
}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants