[
https://issues.apache.org/jira/browse/THRIFT-2858?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14628199#comment-14628199
]
ASF GitHub Bot commented on THRIFT-2858:
----------------------------------------
GitHub user ben-craig opened a pull request:
https://github.com/apache/thrift/pull/552
THRIFT-2858 making HTTP case insensitivity build on Windows
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/ben-craig/thrift HTTPCase
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/thrift/pull/552.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #552
----
commit 20279078199fc35c40d4a2463afa66ea2b79b829
Author: ben-craig <[email protected]>
Date: 2015-07-15T15:04:54Z
Windows build works now
----
> 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
> Assignee: Roger Meier
> 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)
