-
Notifications
You must be signed in to change notification settings - Fork 37
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
Serve 404's on / requests for http://ns.test-ns-signed.internet.nl #1175
Labels
Comments
11 tasks
bwbroersma
changed the title
Do not return website on http://ns.test-ns-signed.internet.nl
Serve 404's on non /connection/ requests to http://*.test-ns-signed.internet.nl
Nov 14, 2023
If we add a wildcard cert on |
bwbroersma
changed the title
Serve 404's on non /connection/ requests to http://*.test-ns-signed.internet.nl
Serve 404's on / requests for http://ns.test-ns-signed.internet.nl
Nov 14, 2023
bwbroersma
added a commit
to bwbroersma/Internet.nl
that referenced
this issue
Nov 14, 2023
Partly fixes internetstandards#1175 Signed-off-by: Benjamin W. Broersma <[email protected]>
bwbroersma
added a commit
to bwbroersma/Internet.nl
that referenced
this issue
Nov 14, 2023
Partly fixes internetstandards#1175 Signed-off-by: Benjamin W. Broersma <[email protected]>
bwbroersma
added a commit
to bwbroersma/Internet.nl
that referenced
this issue
Nov 14, 2023
Partly fixes internetstandards#1175 Signed-off-by: Benjamin W. Broersma <[email protected]>
bwbroersma
added a commit
to bwbroersma/Internet.nl
that referenced
this issue
Nov 14, 2023
Partly fixes internetstandards#1175 Signed-off-by: Benjamin W. Broersma <[email protected]>
mxsasha
pushed a commit
to bwbroersma/Internet.nl
that referenced
this issue
Nov 15, 2023
Partly fixes internetstandards#1175 Signed-off-by: Benjamin W. Broersma <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Do not return website on http://ns.test-ns-signed.internet.nl
Somehow it returns a 200 here, while
curl internet.nl -H 'Host: random.internet.nl'
or the direct IP do return a 404, it's just ns.test-ns-signed.internet.nl that seems to work.Update: this is needed for the connection test
Internet.nl/docker/webserver/nginx_templates/app.conf.template
Lines 88 to 113 in 80ff426
However, the/|
on line 101 can be removed, so this would result in a 404, except for the XHR requests.Update2: the connection test actually uses both paths:
http://[0-9a-f]{32}.{bogus,aaaa,a}.conn.test-ns-signed.internet.nl/?callback=
andhttp://[0-9a-f]{32}.a-aaaa.conn.test-ns6-signed.internet.nl/?callback=
andhttp://[ipv6]/connection/addr-test/[0-9a-f]{32}/?callback=
See:
Internet.nl/frontend/js/internetnl.connection.js
Lines 50 to 54 in 80ff426
The text was updated successfully, but these errors were encountered: