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

ENDOC-535-ssl-note #546

Merged
merged 3 commits into from
Aug 4, 2022
Merged

ENDOC-535-ssl-note #546

merged 3 commits into from
Aug 4, 2022

Conversation

jyunmitch
Copy link
Collaborator

No description provided.

@jyunmitch jyunmitch requested review from nshaw and Lyd1aCla1r3 August 3, 2022 19:22
@@ -140,6 +140,14 @@ Now let's create a new application similar to Scenario 3, but with the self-sign

Once the deployment is complete, you can confirm that all routes use HTTPS with the self-signed certificate. You may still see security warnings in the browser.

#### Note: Multiple Certificate Chains
If you have multiple certificates to load into Openshift, you must pass each certificate with its own identifier.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OpenShift

#### Note: Multiple Certificate Chains
If you have multiple certificates to load into Openshift, you must pass each certificate with its own identifier.
```bash
ca0.crt: -\> a single ca certificate
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CA

@@ -140,6 +140,14 @@ Now let's create a new application similar to Scenario 3, but with the self-sign

Once the deployment is complete, you can confirm that all routes use HTTPS with the self-signed certificate. You may still see security warnings in the browser.

#### Note: Multiple Certificate Chains
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  1. A certificate chain by definition involves multiple certificates so we can simplify the title to Certificate Chain.
  2. Maybe we should either move this note up to step 4, or connect to it here as a foot note. Otherwise they may do the whole thing and then only see this note at the bottom after it fails.

@jyunmitch jyunmitch merged commit 562f390 into main Aug 4, 2022
@jyunmitch jyunmitch deleted the ENDOC-535-ssl-note branch August 4, 2022 13:10
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.

3 participants