Skip to content

Update 'Content-Length' value for fix characters missing - #64

Closed
EthianWong wants to merge 2 commits into
exceptionless:masterfrom
EthianWong:master
Closed

Update 'Content-Length' value for fix characters missing#64
EthianWong wants to merge 2 commits into
exceptionless:masterfrom
EthianWong:master

Conversation

@EthianWong

Copy link
Copy Markdown
Contributor

In other languages (Chinese , Japanese .... )
One word maybe use two or three bytes ...

var a = "我爱你";
console.log(a.length); //3
console.log(new Buffer(a).length); //9

So you should use new Buffer for set "Content-Length" and this problem will be resolve

@CLAassistant

CLAassistant commented Nov 10, 2016

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@niemyjski

Copy link
Copy Markdown
Member

Thanks a ton, I'll review these changes tomorrow!

@EthianWong EthianWong closed this Nov 11, 2016
@niemyjski

Copy link
Copy Markdown
Member

This is a duplicate of #65

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants