Bug #118425 for Hijk: Headers should be case-insensitive

Tue Oct 18 08:28:35 2016 develop [...] traveljury.com - Ticket created

Subject:

Headers should be case-insensitive

Headers are checked case-sensitively here https://metacpan.org/source/AVAR/Hijk-0.26/lib/Hijk.pm#L86 According to https://www.w3.org/Protocols/rfc2616/rfc2616-sec4.html#sec4.2 "Field names are case-insensitive." This code breaks in Elasticsearch 5.0 which now returns lowercase headers, and a request from Hijk just hangs until it times out.

Thu Oct 27 10:16:48 2016 GUGOD [...] cpan.org - Correspondence added

在 2016-十月-18 14:28:35 星期二時，DRTECH 寫到： Show quoted text

> Headers are checked case-sensitively here > https://metacpan.org/source/AVAR/Hijk-0.26/lib/Hijk.pm#L86 According > to https://www.w3.org/Protocols/rfc2616/rfc2616-sec4.html#sec4.2 > "Field names are case-insensitive." > > This code breaks in Elasticsearch 5.0 which now returns lowercase > headers, and a request from Hijk just hangs until it times out.

Thanks for the nice info Clinton, A patch is pushed to address this issue, https://github.com/gugod/Hijk/commit/2d410c3f0fa5b4bb568dd3fe15f72ffb2c9027c1 I don't expect it breaks any existing stuff, but I'll briefly test it and release it.

Thu Oct 27 10:16:49 2016 The RT System itself - Status changed from 'new' to 'open'

Fri Oct 28 07:43:55 2016 GUGOD [...] cpan.org - Taken

Fri Oct 28 07:45:53 2016 GUGOD [...] cpan.org - Correspondence added

在 2016-十月-27 16:16:48 星期四時，GUGOD 寫到： Show quoted text

> A patch is pushed to address this issue, > https://github.com/gugod/Hijk/commit/2d410c3f0fa5b4bb568dd3fe15f72ffb2c9027c1 > > I don't expect it breaks any existing stuff, but I'll briefly test it > and release it.

Hijk 0.27 was just released with only this change. I briefly tested with Elasticsearch 5.0 and that looks OK. If it all looks OK on your side as well feel free to close this ticket :)

Wed Nov 02 07:50:22 2016 develop [...] traveljury.com - Correspondence added

Thanks gugod - looks good to me closing

Wed Nov 02 07:50:23 2016 develop [...] traveljury.com - Status changed from 'open' to 'resolved'