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

[FAB-18298] Default cluster cert and key #2078

Merged
merged 1 commit into from
Nov 6, 2020

Conversation

stephyee
Copy link
Contributor

@stephyee stephyee commented Nov 5, 2020

Type of change

  • New feature

Description

When cluster listener is reused, default cluster client certificate and private key (if not set) to the TLS server certificate and key.

Related issues

FAB-18298

@stephyee stephyee marked this pull request as ready for review November 5, 2020 20:27
@stephyee stephyee requested a review from a team as a code owner November 5, 2020 20:27
@stephyee stephyee marked this pull request as draft November 5, 2020 22:37
Signed-off-by: Tiffany Harris <tiffany.harris@ibm.com>
Signed-off-by: Will Lahti <wtlahti@us.ibm.com>
@stephyee stephyee marked this pull request as ready for review November 6, 2020 02:07
@lindluni lindluni merged commit d620ab3 into hyperledger:master Nov 6, 2020
@stephyee stephyee deleted the fab-18298 branch November 6, 2020 13:05
@jyellick
Copy link
Contributor

jyellick commented Nov 6, 2020

@pamandrejko @joealewine @stephyee This should probably require a doc update (and should simplify our tutorials since it's a couple parameters that don't need to be specified).

@stephyee
Copy link
Contributor Author

stephyee commented Nov 6, 2020

@jyellick Pam did note that in the JIRA. Should we also backport to 2.2?

@caod123
Copy link

caod123 commented Nov 13, 2020

@Mergifyio backport release-2.2

@mergify
Copy link

mergify bot commented Nov 13, 2020

Command backport release-2.2: success

Backports have been created

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants