-
Notifications
You must be signed in to change notification settings - Fork 5.3k
Use ordinal comparison for known header values #64702
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
Use ordinal comparison for known header values #64702
Conversation
|
Tagging subscribers to this area: @dotnet/ncl Issue DetailsFixes #64053 I ran this code with this patch, and these are the headers from ~14k sites that responded ListAfter making This change can theoretically break someone if they were checking for a given value without
|
Fixes #64053
I ran this code with this patch, and these are the headers from ~14k sites that responded
(only showing those that repeated at least 20 times):
List
After making
Content-Typematching case-sensitive, I added backtext/html; charset=UTF-8as the only common case that we would previously match (e8f4850).This change can theoretically break someone if they were checking for a given value without
IgnoreCase, the server responded in a different casing and we no longer normalized it to a lowercase representation.