FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

crypto: add OP flag constants added in OpenSSL v1.1.1 by mkrawczuk · Pull Request #33929 · nodejs/node · GitHub

/ node Public

crypto: add OP flag constants added in OpenSSL v1.1.1 - #33929

Closed
mkrawczuk wants to merge 2 commits into
nodejs:masterfrom
mkrawczuk:sslopts
ConversationCommits (2)ChecksFiles changed
Closed

crypto: add OP flag constants added in OpenSSL v1.1.1#33929
mkrawczuk wants to merge 2 commits into
nodejs:masterfrom
mkrawczuk:sslopts

Conversation

mkrawczuk commented Jun 17, 2020

Copy link
Copy Markdown
Contributor
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • documentation is changed or added
  • commit message follows [commit guidelines]

nodejs-github-bot added the c++ Issues and PRs that require attention from people who are familiar with C++. label Jun 17, 2020

mkrawczuk commented Jun 17, 2020

Copy link
Copy Markdown
Contributor Author

Full OpenSSL option flag list is available here:
https://wiki.openssl.org/index.php/List_of_SSL_OP_Flags

Also I have noticed that several OpenSSL constants here are missing description:
https://nodejs.org/api/crypto.html#crypto_openssl_options

I think it would be a good idea to add it, or to put a reference to the SSL wiki where the descriptions can be found, or do both. Let me know what you think.

tniessen reviewed Jun 18, 2020
View reviewed changes
Comment thread doc/api/crypto.md Outdated
Comment thread doc/api/crypto.md Outdated

addaleax commented Jun 19, 2020

Copy link
Copy Markdown
Member

@nodejs/crypto

addaleax added the crypto Issues and PRs related to the crypto subsystem. label Jun 19, 2020

nodejs-github-bot commented Jun 22, 2020

Copy link
Copy Markdown
Collaborator

CI: https://ci.nodejs.org/job/node-test-pull-request/32013/

mildsunrise commented Jun 22, 2020

Copy link
Copy Markdown
Contributor

I think it would be a good idea to add it, or to put a reference to the SSL wiki where the descriptions can be found, or do both. Let me know what you think.

+1 on doing both

mkrawczuk added a commit to mkrawczuk/node that referenced this pull request Jun 25, 2020
doc: add a reference to the list of OpenSSL flags.
dece24b 
Some of the SSL_OP_* constants are missing description in the documentation.
Instead of rewriting the description from OpenSSL's wiki, I have decided to put
a link to a detailed list in the 'OpenSSL Options' section.

I see no point of doing both - adding a reference to the wiki and adding
constant descriptions - but I might do if presented with convincing arguments.

This is a follow-up to nodejs#33929.
mkrawczuk mentioned this pull request Jun 25, 2020
Closed
2 tasks
jasnell pushed a commit that referenced this pull request Jul 3, 2020
doc: add a reference to the list of OpenSSL flags.
d3d4ace 
Some of the SSL_OP_* constants are missing description in the
documentation. Instead of rewriting the description from OpenSSL's
wiki, I have decided to put a link to a detailed list in the
'OpenSSL Options' section.

I see no point of doing both - adding a reference to the wiki and
adding constant descriptions - but I might do if presented with
convincing arguments.

This is a follow-up to #33929.

PR-URL: #34050
Reviewed-By: James M Snell <jasnell@gmail.com>
MylesBorins pushed a commit that referenced this pull request Jul 14, 2020
doc: add a reference to the list of OpenSSL flags.
f85c7b7 
Some of the SSL_OP_* constants are missing description in the
documentation. Instead of rewriting the description from OpenSSL's
wiki, I have decided to put a link to a detailed list in the
'OpenSSL Options' section.

I see no point of doing both - adding a reference to the wiki and
adding constant descriptions - but I might do if presented with
convincing arguments.

This is a follow-up to #33929.

PR-URL: #34050
Reviewed-By: James M Snell <jasnell@gmail.com>
MylesBorins pushed a commit that referenced this pull request Jul 16, 2020
doc: add a reference to the list of OpenSSL flags.
1558800 
Some of the SSL_OP_* constants are missing description in the
documentation. Instead of rewriting the description from OpenSSL's
wiki, I have decided to put a link to a detailed list in the
'OpenSSL Options' section.

I see no point of doing both - adding a reference to the wiki and
adding constant descriptions - but I might do if presented with
convincing arguments.

This is a follow-up to #33929.

PR-URL: #34050
Reviewed-By: James M Snell <jasnell@gmail.com>

mkrawczuk commented Aug 5, 2020

Copy link
Copy Markdown
Contributor Author

This PR seems to have gotten a little stuck. Is there anything that should be done to move further?
cc @nodejs/crypto, @jasnell @mildsunrise @tniessen

nodejs-github-bot commented Aug 6, 2020

Copy link
Copy Markdown
Collaborator

CI: https://ci.nodejs.org/job/node-test-pull-request/32644/

nodejs-github-bot commented Aug 6, 2020

Copy link
Copy Markdown
Collaborator

CI: https://ci.nodejs.org/job/node-test-pull-request/32645/

This was referenced Aug 7, 2020
Open
Open

mildsunrise commented Aug 7, 2020

Copy link
Copy Markdown
Contributor

Landed in 3306070

mildsunrise closed this Aug 7, 2020
mildsunrise pushed a commit that referenced this pull request Aug 7, 2020
crypto: add OP flag constants added in OpenSSL v1.1.1
3306070 
PR-URL: #33929
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Alba Mendez <me@alba.sh>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
This was referenced Aug 8, 2020
Open
Open
addaleax pushed a commit that referenced this pull request Aug 8, 2020
crypto: add OP flag constants added in OpenSSL v1.1.1
a7b4318 
PR-URL: #33929
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Alba Mendez <me@alba.sh>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
This was referenced Aug 9, 2020
Open
Open
Open
Open
codebytere mentioned this pull request Aug 10, 2020
Merged
This was referenced Aug 11, 2020
Open
Open
codebytere pushed a commit that referenced this pull request Aug 11, 2020
crypto: add OP flag constants added in OpenSSL v1.1.1
a503470 
PR-URL: #33929
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Alba Mendez <me@alba.sh>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
addaleax pushed a commit that referenced this pull request Sep 22, 2020
doc: add a reference to the list of OpenSSL flags.
8960a63 
Some of the SSL_OP_* constants are missing description in the
documentation. Instead of rewriting the description from OpenSSL's
wiki, I have decided to put a link to a detailed list in the
'OpenSSL Options' section.

I see no point of doing both - adding a reference to the wiki and
adding constant descriptions - but I might do if presented with
convincing arguments.

This is a follow-up to #33929.

PR-URL: #34050
Reviewed-By: James M Snell <jasnell@gmail.com>
addaleax pushed a commit that referenced this pull request Sep 22, 2020
crypto: add OP flag constants added in OpenSSL v1.1.1
c33d607 
PR-URL: #33929
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Alba Mendez <me@alba.sh>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
addaleax pushed a commit that referenced this pull request Sep 22, 2020
crypto: add OP flag constants added in OpenSSL v1.1.1
865f8e8 
PR-URL: #33929
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Alba Mendez <me@alba.sh>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
codebytere mentioned this pull request Sep 28, 2020
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

jasnell jasnell approved these changes
tniessen tniessen approved these changes
+1 more reviewer
mildsunrise mildsunrise approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

c++ Issues and PRs that require attention from people who are familiar with C++. crypto Issues and PRs related to the crypto subsystem.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

Back | FazBrowse Home | New Git URL