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

test: use ciphers supported by shared openssl #14566

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
8 participants
@kapouer
Contributor

kapouer commented Aug 1, 2017

On debian with openssl 1.1 cli, the ciphers used in those tests were
unknown.

test: use ciphers supported by shared openssl
On debian with openssl 1.1 cli, the ciphers used in those tests were
unknown.
@cjihrig

cjihrig approved these changes Aug 1, 2017

@jasnell

jasnell approved these changes Aug 1, 2017

@mscdex mscdex added openssl tls labels Aug 1, 2017

@Trott

This comment has been minimized.

Show comment
Hide comment

Trott added a commit to Trott/io.js that referenced this pull request Aug 7, 2017

test: use ciphers supported by shared OpenSSL
On Debian with OpenSSL 1.1 CLI, the ciphers used in those tests were
unknown.

PR-URL: nodejs#14566
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
@Trott

This comment has been minimized.

Show comment
Hide comment
@Trott

Trott Aug 7, 2017

Member

Landed in a974753. Thanks, @kapouer!

Member

Trott commented Aug 7, 2017

Landed in a974753. Thanks, @kapouer!

@Trott Trott closed this Aug 7, 2017

addaleax added a commit that referenced this pull request Aug 10, 2017

test: use ciphers supported by shared OpenSSL
On Debian with OpenSSL 1.1 CLI, the ciphers used in those tests were
unknown.

PR-URL: #14566
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>

icarter09 added a commit to icarter09/node that referenced this pull request Aug 12, 2017

test: use ciphers supported by shared OpenSSL
On Debian with OpenSSL 1.1 CLI, the ciphers used in those tests were
unknown.

PR-URL: nodejs#14566
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>

@addaleax addaleax referenced this pull request Aug 13, 2017

Merged

v8.4.0 proposal #14811

MylesBorins added a commit that referenced this pull request Sep 19, 2017

test: use ciphers supported by shared OpenSSL
On Debian with OpenSSL 1.1 CLI, the ciphers used in those tests were
unknown.

PR-URL: #14566
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>

MylesBorins added a commit that referenced this pull request Sep 19, 2017

test: use ciphers supported by shared OpenSSL
On Debian with OpenSSL 1.1 CLI, the ciphers used in those tests were
unknown.

PR-URL: #14566
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>

@MylesBorins MylesBorins referenced this pull request Sep 20, 2017

Merged

v6.11.4 proposal #15506

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