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.

Sign up for GitHub

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

Jump to bottom

crypto: fix default encoding of LazyTransform #8611

Closed
wants to merge 2 commits into from
Closed

crypto: fix default encoding of LazyTransform #8611

wants to merge 2 commits into from

Commits on Feb 16, 2017

  1. crypto: fix default encoding of LazyTransform 

    Change the default encoding from latin1 to utf8
and extend the test-crypto tests.
    Lukas Möller committed Feb 16, 2017
    Configuration menu
    Copy the full SHA
    74b56cd View commit details
    Browse the repository at this point in the history

  2. crypto: fix default encoding of LazyTransform 

    PullRequest nodejs#5522 and nodejs#5500 described the change
of the default encoding into UTF8 in crypto functions.

This however was only changed for the non-streaming API.
The streaming API still used binary as the default encoding.

This commit will change the default streaming API encoding to UTF8
to make both APIs behave the same.

It will also add tests to validate the behavior.
    Lukas Möller committed Feb 16, 2017
    Configuration menu
    Copy the full SHA
    fe9de5d View commit details
    Browse the repository at this point in the history