You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@thrift.apache.org by "Claudius Heine (JIRA)" <ji...@apache.org> on 2015/07/06 13:22:04 UTC
[jira] [Commented] (THRIFT-2858) Enable header field case
insensitive match in THttpServer
[ https://issues.apache.org/jira/browse/THRIFT-2858?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14614891#comment-14614891 ]
Claudius Heine commented on THRIFT-2858:
----------------------------------------
Patch applies, compiles and "make cross" & "make check" do not have any unexpected errors.
Ready to merge I would say.
> Enable header field case insensitive match in THttpServer
> ---------------------------------------------------------
>
> Key: THRIFT-2858
> URL: https://issues.apache.org/jira/browse/THRIFT-2858
> Project: Thrift
> Issue Type: Bug
> Components: C++ - Library
> Affects Versions: 0.9.1, 0.9.2
> Environment: ubuntu 14.04 LTS
> gcc 4.8
> Reporter: YI-HUNG JEN
> Labels: patch
> Fix For: 0.9.3
>
> Attachments: thrift-2858-enable-THttpServer_cpp-header-case-insensitive-match.patch
>
> Original Estimate: 1h
> Remaining Estimate: 1h
>
> Communications between THttpServer in cpp lib and nodejs http client caused unexpected ECONNRESET. This was caused by nodejs client setting "Content-length" instead of "Content-Length" in the header field, causing THttpServer to terminate the request prematurely.
> Per HTTP specification, header field name is case insensitive, therefore this patch used case insensitive match to identify "Content-Length" field.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)