-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
Cache and case in parameter Url #10395
Comments
OK, so I tried to reproduce this issue. Tried it in firefox - works as expected. Tried it with curl - works as expected. Tried it in Chromium. If I load https://img.shields.io/badge/logo-BugBugBugBug-blue?logo=javascript&logocolor=red in one tab, and then open another tab and try and type https://img.shields.io/badge/logo-BugBugBugBug-blue?logo=javascript&logoColor=red in another tab, it seems to manually change the URL back to https://img.shields.io/badge/logo-BugBugBugBug-blue?logo=javascript&logocolor=red - its not even considering If you go back and try it, is this what you're actually seeing? This seems like weird browser behaviour rather than anything on our side. |
I'd agree with this assessment and will add some content below that I think supports this (emphasis mine): https://www.rfc-editor.org/rfc/rfc3986#section-6.2.2.1
https://www.rfc-editor.org/rfc/rfc7230#section-2.7.3
|
Yes, it's seems it's a chromium issues, thanks for the analysis. I think we can close this issue ? |
agreed, thanks for reaching out! |
Are you experiencing an issue with...
shields.io
🐞 Description
The cache remember the case in url, if one parameter is written with wrong case, it will break this parameter until cache expire (i assume this is a cache issue but I might be wrong).
Here an example:
https://img.shields.io/badge/logo-BugBugBugBug-blue?logo=javascript&logocolor=red
logoColor
case is not good, the c is in lowercase instead of upper case.https://img.shields.io/badge/logo-BugBugBugBug-blue?logo=javascript&logoColor=red
edit : for some reason the bug doesn't happen if I make clickable link. So you now need to copy/paste Url.
🔗 Link to the badge
No response
💡 Possible Solution
No response
The text was updated successfully, but these errors were encountered: