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

MagicDNS now seems to be required for go-links to work #102

Closed
Tuinslak opened this issue Dec 20, 2023 · 0 comments · Fixed by #103
Closed

MagicDNS now seems to be required for go-links to work #102

Tuinslak opened this issue Dec 20, 2023 · 0 comments · Fixed by #103

Comments

@Tuinslak
Copy link

With MagicDNS disabled:

$ dig go +short
100.99.0.1

$ curl go/ -I
curl: (28) Failed to connect to go port 80 after 75437 ms: Couldn't connect to server

With MagicDNS enabled (and a restart):

$ curl go/ -I
НТТР/1.1 302 Found
Content-Type: text/html; charset=utf-8
Location: https:/go.bunny-major.ts.net/
Date: Tue, 19 Dec 2023 18:06:27 GMT

НТТР/1.1 200 0K
Date: Tue, 19 Dec 2023 18:06:27 GMT

The docker logs say:

2023/12/19 18:01:11 tsnet: you must enable HTTPS in the admin panel to proceed. 

Entire log after a restart with MagicDNS disabled: https://pastebin.com/921CSuZZ (can't include it here -- too long)

patrickod added a commit that referenced this issue Dec 20, 2023
Fixes #102

Ensure that we have CertDomains for the HTTPS listener before asking
tsnet to create one.
patrickod added a commit that referenced this issue Dec 20, 2023
Fixes #102

Ensure that we have CertDomains for the HTTPS listener before asking
tsnet to create one.

Signed-off-by: Patrick O'Doherty <[email protected]>
patrickod added a commit that referenced this issue Dec 20, 2023
Fixes #102

Ensure that we have CertDomains for the HTTPS listener before asking
tsnet to create one.

Signed-off-by: Patrick O'Doherty <[email protected]>
patrickod added a commit that referenced this issue Dec 20, 2023
Fixes #102

Ensure that we have CertDomains for the HTTPS listener before asking
tsnet to create one.

Signed-off-by: Patrick O'Doherty <[email protected]>
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 a pull request may close this issue.

1 participant