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

Fails for URLs without protocol, but with path #168

Open
dagilleland opened this issue Feb 3, 2025 · 1 comment
Open

Fails for URLs without protocol, but with path #168

dagilleland opened this issue Feb 3, 2025 · 1 comment
Labels
enhancement Improves an existing feature. p3 We don't have visibility when this will be addressed.

Comments

@dagilleland
Copy link

What information was incorrect, unhelpful, or incomplete?

GitHub pages, such as this one dgilleland.github.io/CPSC-1520 may have case-sensitive urls. When testing in the Observatory, it transforms the supplied URL to lower-case, resulting in a 404 from github.io.

Image

What did you expect to see?

Some analysis of my site, but not an error saying the URL is incorrect/not resolved.

Do you have any supporting links, references, or citations?

My repository is public: https://github.com/dgilleland/CPSC-1520

Do you have anything more you want to share?

No response

@dagilleland dagilleland added the needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened. label Feb 3, 2025
@caugner
Copy link
Contributor

caugner commented Feb 5, 2025

Note that HTTP Observatory on MDN scans hosts, not individual URLs, and if you provide a valid URL, the host will generally be extracted automatically. So typing these is equivalent:

  • dgilleland.github.io
  • https://dgilleland.github.io/CPSC-1520

Note that dgilleland.github.io/CPSC-1520 is not a valid fully qualified URL, and therefore HTTP Observatory seems to treat it as a host.

It would probably makes sense to remove the /... part automatically, but since you would usually type in a host or paste in a fully qualified URL, and this is the workaround, this has rather low priority.

@caugner caugner added enhancement Improves an existing feature. p3 We don't have visibility when this will be addressed. and removed needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened. labels Feb 5, 2025
@caugner caugner changed the title Fails with case-sensitive URLs Fails for URLs without protocol, but with path Feb 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Improves an existing feature. p3 We don't have visibility when this will be addressed.
Projects
None yet
Development

No branches or pull requests

2 participants