masaori335 commented on a change in pull request #7886:
URL: https://github.com/apache/trafficserver/pull/7886#discussion_r651390093
##########
File path: proxy/hdrs/HdrToken.cc
##########
@@ -548,6 +548,30 @@ hdrtoken_tokenize_dfa(const char *string, int string_len,
const char **wks_strin
return wks_idx;
}
+/*-------------------------------------------------------------------------
+ Have to work around that methods are case insensitive while the DFA is
+ case insensitive.
+ -------------------------------------------------------------------------*/
+
+int
+hdrtoken_method_tokenize(const char *string, int string_len)
Review comment:
Note: performance perspective, it looks like this has some overhead, but
it might not be matter. Because `GET` is processed in the fast path and doesn't
come here.
https://github.com/apache/trafficserver/blob/f002cdbae85509f339f020051d68f8cecdf17bd1/proxy/hdrs/HTTP.cc#L935-L940
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
