-
Notifications
You must be signed in to change notification settings - Fork 38
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
Show full URL of retrieved security.txt file instead of just host #1314
Comments
An extra enhancement would be to see the complete redirect path, but just showing the full resolved path of the sectxt would already indeed be very useful. |
I think I have a patch. Note that is will only update the 'retrieved from', not the 'requested from', since the last one will likely otherwise show Internet.nl/checks/tasks/securitytxt.py Lines 46 to 47 in d0163da
Some samples of 'Technical details' (of rijksoverheid.nl and example.org):
Another thing is this would break all old reports, since they don't have the Maybe the dot should be removed behind the url for easier copying? It's still there because I just changed the |
True, though we could preserve the original path. I think this approach is reasonable too.
Yes, this backfill is a good approach.
We should remove the dot, yes |
The explanation for security.txt may show a URL that you can copy and paste to avoid typing problems.
See an incomplete path for example in:
https://en.internet.nl/site/rijksoverheid.nl/2671942/#control-panel-31
currently: security.txt retrieved from www.ncsc.nl.
proposed: (security.txt retrieved from) https://www.ncsc.nl/.well-known/security.txt
The text was updated successfully, but these errors were encountered: