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

doc: correct crypto encoding information #7805

Closed
wants to merge 1 commit into from
Closed

doc: correct crypto encoding information #7805

wants to merge 1 commit into from

Conversation

hugnosis
Copy link
Contributor

Checklist
  • make -j4 test (UNIX), or vcbuild test nosign (Windows) passes
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows commit guidelines
Affected core subsystem(s)

doc

Description of change

The default encoding for crypto methods was changed in v6.0.0, with v4.x keeping a default of binary.

Ref: b010c87

The default encoding for crypto methods was changed in v6.0.0, with v4.x keeping a default of binary.
@nodejs-github-bot nodejs-github-bot added crypto Issues and PRs related to the crypto subsystem. doc Issues and PRs related to the documentations. labels Jul 20, 2016
@hugnosis hugnosis changed the title Update crypto.md doc: correct crypto encoding information Jul 20, 2016
@claudiorodriguez
Copy link
Contributor

LGTM

@thefourtheye
Copy link
Contributor

cc @nodejs/crypto

@indutny
Copy link
Member

indutny commented Jul 20, 2016

Do we have a test for this?

@jasnell
Copy link
Member

jasnell commented Jul 26, 2016

LGTM

@jasnell
Copy link
Member

jasnell commented Aug 1, 2016

ping @nodejs/crypto
@indutny ... not certain about the tests, but given that this is a doc change, tests can be added if necessary separately

jasnell pushed a commit that referenced this pull request Aug 4, 2016
The default encoding for crypto methods was changed in v6.0.0
with v4.x keeping a default of binary.

PR-URL: #7805
Reviewed-By: Claudio Rodriguez <cjrodr@yahoo.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
@jasnell
Copy link
Member

jasnell commented Aug 4, 2016

Landed in 561958e!

@jasnell jasnell closed this Aug 4, 2016
@cjihrig cjihrig mentioned this pull request Aug 8, 2016
cjihrig pushed a commit that referenced this pull request Aug 10, 2016
The default encoding for crypto methods was changed in v6.0.0
with v4.x keeping a default of binary.

PR-URL: #7805
Reviewed-By: Claudio Rodriguez <cjrodr@yahoo.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
@cjihrig cjihrig mentioned this pull request Aug 11, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
crypto Issues and PRs related to the crypto subsystem. doc Issues and PRs related to the documentations.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants