Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Handle UTF-8 multibyte sequences in node #13

Merged
merged 1 commit into from Sep 21, 2014

Conversation

@mathbruyen
Copy link

commented Sep 21, 2014

Content-length header was set to the length of the string, which does not take into account multibyte sequences in UTF-8 strings. Now converts first to a buffer to get the proper length.

Mathieu Bruyen
Handle UTF-8 multibyte sequences in node
Content-length header was set to the length of the string, which
does not take into account multibyte sequences in UTF-8 strings.

Now converts first to a buffer to get the proper length.

mzabriskie added a commit that referenced this pull request Sep 21, 2014

Merge pull request #13 from mathbruyen/utf8_content_length
Handle UTF-8 multibyte sequences in node

@mzabriskie mzabriskie merged commit c0201fd into axios:master Sep 21, 2014

1 check passed

continuous-integration/travis-ci The Travis CI build passed
Details

@mathbruyen mathbruyen deleted the mathbruyen:utf8_content_length branch Sep 21, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.