[ 
https://issues.apache.org/jira/browse/THRIFT-2998?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16220634#comment-16220634
 ] 

James E. King, III commented on THRIFT-2998:
--------------------------------------------

While this change looks good, when I enable cross testing of nodejs and http 
transport, some things pass and some do not.  Too many do not...  *grumble*

> Node.js: Missing header from http request
> -----------------------------------------
>
>                 Key: THRIFT-2998
>                 URL: https://issues.apache.org/jira/browse/THRIFT-2998
>             Project: Thrift
>          Issue Type: Bug
>          Components: Node.js - Library
>    Affects Versions: 0.9.2
>            Reporter: Traian Stanev
>            Assignee: James E. King, III
>            Priority: Minor
>
> The "Content-Type" header is not being set to "application/x-thrift" when 
> making http requests in the node.js implementation. This results in error 
> response (status 400) for any such connections (depending on whether the 
> server checks the header). The Python implementation for example adds the 
> header explicitly:
> https://github.com/tstanev/thrift/blob/master/lib/py/src/transport/THttpClient.py#L127



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to