Opened 5 years ago
Last modified 5 years ago
#1500 accepted enhancement
ngx_hash_t can have only lower case key
|Reported by:||Owned by:|
|uname -a:||16.7.0 Darwin Kernel Version 16.7.0: Thu Jan 11 22:59:40 PST 2018; root:xnu-3789.73.8~1/RELEASE_X86_64 x86_64|
ngx_hash_init convert all the keys in lower case, so when use ngx_hash_find it returns null.
Below is the code line in ngx_hash.c.
key = ngx_hash(key, ngx_tolower(data[i]));
I think, you can make it generic which supports case sensitive keys.
Change History (2)
comment:1 by , 5 years ago
|Status:||new → accepted|
|Type:||defect → enhancement|
comment:2 by , 5 years ago
I think you should mention in the documentation that ngx_hash_t doesnt support case sensitive keys. And in my case it was needed and I just use another function to copy string instead of ngx_tolower. I hope it doesnt break the hash?
Note: See TracTickets for help on using tickets.
yes, you are right, currently only lowercase keys are possible.
there is no great need in case sensitive keys though at the moment.