mirrors/haproxy
1
0
Fork 0
mirror of https://git.haproxy.org/git/haproxy.git/ synced 2025-09-25 15:51:24 +02:00
Code Issues Projects Releases Wiki Activity

MINOR: htx: switch to case sensitive search of lower case header names

Browse Source 
Now that we know that htx only contains lower case header names, there
is no need anymore for looking them up in a case-insensitive manner.

Note that http_find_header() still does it because header names to
compare against may come from everywhere there.
This commit is contained in:
Willy Tarreau 2018-12-07 11:38:03 +01:00
parent d6735d611e
commit 2e754bff23
1 changed files with 5 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
src/proto_htx.c
View File

@ -4609,7 +4609,7 @@ void htx_check_request_for_cacheability(struct stream *s, struct channel *req)
n = htx_get_blk_name(htx, blk);
v = htx_get_blk_value(htx, blk);
if (isteqi(n, ist("Pragma"))) {
if (isteq(n, ist("pragma"))) {
if (v.len >= 8 && strncasecmp(v.ptr, "no-cache", 8) == 0) {
pragma_found = 1;
continue;
@ -4618,13 +4618,13 @@ void htx_check_request_for_cacheability(struct stream *s, struct channel *req)
/* Don't use the cache and don't try to store if we found the
* Authorization header */
if (isteqi(n, ist("Authorization"))) {
if (isteq(n, ist("authorization"))) {
txn->flags &= ~TX_CACHEABLE & ~TX_CACHE_COOK;
txn->flags |= TX_CACHE_IGNORE;
continue;
}
if (!isteqi(n, ist("Cache-control")))
if (!isteq(n, ist("cache-control")))
continue;
/* OK, right now we know we have a cache-control header */
@ -4697,14 +4697,14 @@ void htx_check_response_for_cacheability(struct stream *s, struct channel *res)
n = htx_get_blk_name(htx, blk);
v = htx_get_blk_value(htx, blk);
if (isteqi(n, ist("Pragma"))) {
if (isteq(n, ist("pragma"))) {
if ((v.len >= 8) && strncasecmp(v.ptr, "no-cache", 8) == 0) {
txn->flags &= ~TX_CACHEABLE & ~TX_CACHE_COOK;
return;
}
}
if (!isteqi(n, ist("Cache-control")))
if (!isteq(n, ist("cache-control")))
continue;
/* OK, right now we know we have a cache-control header */