Update certs for SecuritySSLReloadCertsActionTests #2679

cwperks · 2023-04-13T22:17:57Z

Description

This PR updates expired certs used in SecuritySSLReloadCertsActionTests

Category (Enhancement, New feature, Bug fix, Test fix, Refactoring, Maintenance, Documentation)

Test fix

Issues Resolved

#2675

Check List

New functionality includes testing
New functionality has been documented
Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Signed-off-by: Craig Perkins <[email protected]>

cwperks · 2023-04-13T22:20:03Z

src/test/java/org/opensearch/security/ssl/SecuritySSLReloadCertsActionTests.java

@@ -52,18 +52,18 @@ public class SecuritySSLReloadCertsActionTests extends SingleClusterTest {
        ImmutableMap.of(
        "issuer_dn", "CN=Example Com Inc. Signing CA,OU=Example Com Inc. Signing CA,O=Example Com Inc.,DC=example,DC=com",
        "subject_dn", "CN=node-1.example.com,OU=SSL,O=Test,L=Test,C=DE",
-            "san", "[[8, 1.2.3.4.5.5], [0, [2.5.4.3, node-1.example.com]], [2, node-1.example.com], [2, localhost], [7, 127.0.0.1]]",


I changed this back from the change introduced in this PR because I'm not positive how to specify the SAN in the openssl command. PR where this was added: https://github.com/opensearch-project/security/pull/1369/files#diff-9e1e704af587fb1787556fe19857df59ef099619245b491958b0c9de83e5f2fe

Node certificates are being generated with the following commands:

openssl genrsa -out node-key-temp.pem 2048 openssl pkcs8 -inform PEM -outform PEM -in node-key-temp.pem -topk8 -nocrypt -v1 PBE-SHA1-3DES -out node.key.pem openssl req -new -key node.key.pem -subj "/C=DE/L=Test/O=Test/OU=SSL/CN=node-1.example.com" -out node.csr openssl x509 -req -days 3650 -extfile <(printf "subjectAltName=DNS:node-1.example.com,DNS:localhost,IP:127.0.0.1,RID:1.2.3.4.5.5") -in node.csr -out node.crt.pem -CA signing.pem -CAkey signing-key.pem

I'm able to add otherName in the subjectAltName, but I'm not sure if this is correct:

openssl genrsa -out node2-key-temp.pem 2048 openssl pkcs8 -inform PEM -outform PEM -in node2-key-temp.pem -topk8 -nocrypt -v1 PBE-SHA1-3DES -out node2.key.pem openssl req -new -key node2.key.pem -subj "/C=DE/L=Test/O=Test/OU=SSL/CN=node-2.example.com" -out node2.csr openssl x509 -req -days 3650 -extfile <(printf "subjectAltName=DNS:node-2.example.com,DNS:localhost,IP:127.0.0.1,RID:1.2.3.4.5.5,otherName:2.5.4.3;UTF8:node-2.example.com") -in node2.csr -out node2.crt.pem -CA signing.pem -CAkey signing-key.pem

@jksmth Would you be able to provide the commands you used to generate certs in this PR? #1369

I regenerated the node certificates using:

openssl genrsa -out node-key-temp.pem 2048 openssl pkcs8 -inform PEM -outform PEM -in node-key-temp.pem -topk8 -nocrypt -v1 PBE-SHA1-3DES -out node.key.pem openssl req -new -key node.key.pem -subj "/C=DE/L=Test/O=Test/OU=SSL/CN=node-1.example.com" -out node.csr openssl x509 -req -days 3650 -extfile <(printf "subjectAltName=RID:1.2.3.4.5.5,otherName:2.5.4.3;UTF8:node-1.example.com,DNS:node-1.example.com,DNS:localhost,IP:127.0.0.1") -in node.csr -out node.crt.pem -CA signing.pem -CAkey signing-key.pem openssl genrsa -out node-new-key-temp.pem 2048 openssl pkcs8 -inform PEM -outform PEM -in node-new-key-temp.pem -topk8 -nocrypt -v1 PBE-SHA1-3DES -out node-new.key.pem openssl req -new -key node-new.key.pem -subj "/C=DE/L=Test/O=Test/OU=SSL/CN=node-1.example.com" -out node-new.csr openssl x509 -req -days 3650 -extfile <(printf "subjectAltName=RID:1.2.3.4.5.5,otherName:2.5.4.3;UTF8:node-1.example.com,DNS:node-1.example.com,DNS:localhost,IP:127.0.0.1") -in node-new.csr -out node-new.crt.pem -CA signing.pem -CAkey signing-key.pem openssl genrsa -out node-wrong-key-temp.pem 2048 openssl pkcs8 -inform PEM -outform PEM -in node-wrong-key-temp.pem -topk8 -nocrypt -v1 PBE-SHA1-3DES -out node-wrong.key.pem openssl req -new -key node-wrong.key.pem -subj "/C=DE/L=Test/O=Test/OU=SSL/CN=node-2.example.com" -out node-wrong.csr openssl x509 -req -days 3650 -extfile <(printf "subjectAltName=RID:1.2.3.4.5.5,otherName:2.5.4.3;UTF8:node-2.example.com,DNS:node-2.example.com,DNS:localhost,IP:127.0.0.1") -in node-wrong.csr -out node-wrong.crt.pem -CA signing.pem -CAkey signing-key.pem

and the tests pass locally

More details about subjectAltName can be found here: https://www.openssl.org/docs/man1.0.2/man5/x509v3_config.html

codecov-commenter · 2023-04-13T22:54:26Z

Codecov Report

Merging #2679 (0d32ae1) into main (9fca0da) will decrease coverage by 0.05%.
The diff coverage is n/a.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@             Coverage Diff              @@
##               main    #2679      +/-   ##
============================================
- Coverage     61.39%   61.35%   -0.05%     
  Complexity     3383     3383              
============================================
  Files           269      269              
  Lines         18668    18668              
  Branches       3279     3279              
============================================
- Hits          11462    11454       -8     
- Misses         5611     5618       +7     
- Partials       1595     1596       +1

see 2 files with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Signed-off-by: Craig Perkins <[email protected]>

cwperks · 2023-04-14T14:07:23Z

I used the commands below to generate new certs:

# openssl req -x509 -sha256 -new -nodes -key rootCAKey.pem -subj "/CN=Example Com Inc. Signing CA/OU=Example Com Inc. Signing CA/O=Example Com Inc./DC=example/DC=com" -days 3650 -out rootCACert.pem


openssl genrsa -out root-ca-key.pem 2048
openssl req -x509 -sha256 -new -nodes -key root-ca-key.pem -subj "/DC=com/DC=example/O=Example Com Inc./OU=Example Com Inc. Root CA/CN=Example Com Inc. Root CA" -days 3650 -out root-ca.pem
openssl genrsa -out signing-key.pem 2048
openssl req -x509 -sha256 -new -nodes -CA root-ca.pem -CAkey root-ca-key.pem -key signing-key.pem -subj "/DC=com/DC=example/O=Example Com Inc./OU=Example Com Inc. Signing CA/CN=Example Com Inc. Signing CA" -days 3650 -out signing.pem

openssl genrsa -out node-key-temp.pem 2048
openssl pkcs8 -inform PEM -outform PEM -in node-key-temp.pem -topk8 -nocrypt -v1 PBE-SHA1-3DES -out node.key.pem
openssl req -new -key node.key.pem -subj "/C=DE/L=Test/O=Test/OU=SSL/CN=node-1.example.com" -out node.csr
openssl x509 -req -days 3650 -extfile <(printf "subjectAltName=RID:1.2.3.4.5.5,otherName:2.5.4.3;UTF8:node-1.example.com,DNS:node-1.example.com,DNS:localhost,IP:127.0.0.1") -in node.csr -out node.crt.pem -CA signing.pem -CAkey signing-key.pem

openssl genrsa -out node-new-key-temp.pem 2048
openssl pkcs8 -inform PEM -outform PEM -in node-new-key-temp.pem -topk8 -nocrypt -v1 PBE-SHA1-3DES -out node-new.key.pem
openssl req -new -key node-new.key.pem -subj "/C=DE/L=Test/O=Test/OU=SSL/CN=node-1.example.com" -out node-new.csr
openssl x509 -req -days 3650 -extfile <(printf "subjectAltName=RID:1.2.3.4.5.5,otherName:2.5.4.3;UTF8:node-1.example.com,DNS:node-1.example.com,DNS:localhost,IP:127.0.0.1") -in node-new.csr -out node-new.crt.pem -CA signing.pem -CAkey signing-key.pem

openssl genrsa -out node-wrong-key-temp.pem 2048
openssl pkcs8 -inform PEM -outform PEM -in node-wrong-key-temp.pem -topk8 -nocrypt -v1 PBE-SHA1-3DES -out node-wrong.key.pem
openssl req -new -key node-wrong.key.pem -subj "/C=DE/L=Test/O=Test/OU=SSL/CN=node-2.example.com" -out node-wrong.csr
openssl x509 -req -days 3650 -extfile <(printf "subjectAltName=RID:1.2.3.4.5.5,otherName:2.5.4.3;UTF8:node-2.example.com,DNS:node-2.example.com,DNS:localhost,IP:127.0.0.1") -in node-wrong.csr -out node-wrong.crt.pem -CA signing.pem -CAkey signing-key.pem

# CN=kirk,OU=client,O=client,L=Test,C=DE
openssl genrsa -out kirk-key-temp.pem 2048
openssl pkcs8 -inform PEM -outform PEM -in kirk-key-temp.pem -topk8 -nocrypt -v1 PBE-SHA1-3DES -out kirk.key.pem
openssl req -new -key kirk.key.pem -subj "/C=DE/L=Test/O=client/OU=client/CN=kirk" -out kirk.csr
openssl x509 -req -days 3650 -in kirk.csr -out kirk.crt.pem -CA signing.pem -CAkey signing-key.pem

openssl pkcs12 -export -in kirk.crt.bundle.pem -inkey kirk.key.pem -name kirk > kirk.p12

# CN=spock,OU=client,O=client,L=Test,C=DE
openssl genrsa -out spock-key-temp.pem 2048
openssl pkcs8 -inform PEM -outform PEM -in spock-key-temp.pem -topk8 -nocrypt -v1 PBE-SHA1-3DES -out spock.key.pem
openssl req -new -key spock.key.pem -subj "/C=DE/L=Test/O=client/OU=client/CN=spock" -out spock.csr
openssl x509 -req -days 3650 -in spock.csr -out spock.crt.pem -CA signing.pem -CAkey signing-key.pem

openssl pkcs12 -export -in spock.crt.bundle.pem -inkey spock.key.pem -name spock > spock.p12

In addition to generating node.crt.pem, node-new.crt.pem, node-wrong.crt.pem, kirk.crt.pem and spock.crt.pem I had to include the entire certificate chain in each file including the intermediate signing certificate and root certificate. I was not able to figure out how to automatically add the certificate chain when creating a .pem file. When bundling the certificates together in the same file the root certificate was placed at the bottom and the lowest level certificate on the top.

cwperks · 2023-04-14T14:38:19Z

This SO post describes how to add all certificates to the keystore. https://stackoverflow.com/a/9300727

In summary, the commands for kirk would be:

# kirk.crt.bundle.pem is all certificates with server certificate on the top and root-ca on the bottom
openssl pkcs12 -export -in kirk.crt.bundle.pem -inkey kirk.key.pem -name kirk > kirk.p12

keytool -importkeystore -srckeystore kirk.p12 -destkeystore kirk-keystore.jks -srcstoretype pkcs12 -alias kirk

# Use password changeit for all prompts

# Export the certificate from the newly created keystore
keytool -exportcert -rfc -file kirk-all.pem -keystore kirk-keystore.jks -alias kirk

# Add all certs to kirk-all.pem similar to kirk.crt.bundle.pem

# Import the bundle

keytool -importcert -keystore kirk-keystore.jks -alias kirk -file kirk-all.pem

truststore.jks can be created with:

keytool -import -trustcacerts -file root-ca.pem -alias root-ca -keystore truststore.jks

stephen-crawford

Thank you for helping with this Craig. Looks good to me!

opensearch-trigger-bot · 2023-04-14T18:35:05Z

The backport to 1.x failed:

The process '/usr/bin/git' failed with exit code 1

To backport manually, run these commands in your terminal:

# Fetch latest updates from GitHub
git fetch
# Create a new working tree
git worktree add .worktrees/backport-1.x 1.x
# Navigate to the new working tree
cd .worktrees/backport-1.x
# Create a new branch
git switch --create backport/backport-2679-to-1.x
# Cherry-pick the merged commit of this pull request and resolve the conflicts
git cherry-pick -x --mainline 1 bbd43ec56cf1d7913bbe49547b09558ac37bc89f
# Push it to GitHub
git push --set-upstream origin backport/backport-2679-to-1.x
# Go back to the original working tree
cd ../..
# Delete the working tree
git worktree remove .worktrees/backport-1.x

Then, create a pull request where the base branch is 1.x and the compare/head branch is backport/backport-2679-to-1.x.

opensearch-trigger-bot · 2023-04-14T18:35:06Z

The backport to 1.3 failed:

The process '/usr/bin/git' failed with exit code 1

To backport manually, run these commands in your terminal:

# Fetch latest updates from GitHub
git fetch
# Create a new working tree
git worktree add .worktrees/backport-1.3 1.3
# Navigate to the new working tree
cd .worktrees/backport-1.3
# Create a new branch
git switch --create backport/backport-2679-to-1.3
# Cherry-pick the merged commit of this pull request and resolve the conflicts
git cherry-pick -x --mainline 1 bbd43ec56cf1d7913bbe49547b09558ac37bc89f
# Push it to GitHub
git push --set-upstream origin backport/backport-2679-to-1.3
# Go back to the original working tree
cd ../..
# Delete the working tree
git worktree remove .worktrees/backport-1.3

Then, create a pull request where the base branch is 1.3 and the compare/head branch is backport/backport-2679-to-1.3.

* Update certs for SecuritySSLReloadCertsActionTests Signed-off-by: Craig Perkins <[email protected]> * Add otherName back in Signed-off-by: Craig Perkins <[email protected]> * Ensure files end in new line Signed-off-by: Craig Perkins <[email protected]> --------- Signed-off-by: Craig Perkins <[email protected]> (cherry picked from commit bbd43ec)

* Update certs for SecuritySSLReloadCertsActionTests Signed-off-by: Craig Perkins <[email protected]> * Add otherName back in Signed-off-by: Craig Perkins <[email protected]> * Ensure files end in new line Signed-off-by: Craig Perkins <[email protected]> --------- Signed-off-by: Craig Perkins <[email protected]> (cherry picked from commit bbd43ec) Co-authored-by: Craig Perkins <[email protected]>

…t#2679) * Update certs for SecuritySSLReloadCertsActionTests Signed-off-by: Craig Perkins <[email protected]> * Add otherName back in Signed-off-by: Craig Perkins <[email protected]> * Ensure files end in new line Signed-off-by: Craig Perkins <[email protected]> --------- Signed-off-by: Craig Perkins <[email protected]> Signed-off-by: Maciej Mierzwa <[email protected]>

…t#2679) * Update certs for SecuritySSLReloadCertsActionTests Signed-off-by: Craig Perkins <[email protected]> * Add otherName back in Signed-off-by: Craig Perkins <[email protected]> * Ensure files end in new line Signed-off-by: Craig Perkins <[email protected]> --------- Signed-off-by: Craig Perkins <[email protected]> Signed-off-by: Sam <[email protected]>

Update certs for SecuritySSLReloadCertsActionTests

cbf6121

Signed-off-by: Craig Perkins <[email protected]>

cwperks commented Apr 13, 2023

View reviewed changes

stephen-crawford mentioned this pull request Apr 14, 2023

Update certificates to fix SSL test failures #2678

Closed

3 tasks

Add otherName back in

e443519

Signed-off-by: Craig Perkins <[email protected]>

cwperks marked this pull request as ready for review April 14, 2023 13:30

cwperks requested review from cliu123, DarshitChanpura, davidlago, peternied, RyanL1997, stephen-crawford and reta as code owners April 14, 2023 13:30

Ensure files end in new line

0d32ae1

Signed-off-by: Craig Perkins <[email protected]>

stephen-crawford approved these changes Apr 14, 2023

View reviewed changes

RyanL1997 approved these changes Apr 14, 2023

View reviewed changes

stephen-crawford added backport opendistro-1.3 Backport to opendistro-1.3 branch backport 1.x backport to 1.x branch backport 2.x backport to 2.x branch labels Apr 14, 2023

cwperks added backport 1.3 backport to 1.3 branch and removed backport opendistro-1.3 Backport to opendistro-1.3 branch labels Apr 14, 2023

cwperks merged commit bbd43ec into opensearch-project:main Apr 14, 2023

opensearch-trigger-bot bot mentioned this pull request Apr 14, 2023

[Backport 2.x] Update certs for SecuritySSLReloadCertsActionTests #2682

Merged

stephen-crawford mentioned this pull request Apr 14, 2023

[Backport 1.x] Backport #2679 Update Certs for SSL #2683

Merged

3 tasks

This was referenced Apr 14, 2023

[Backport 1.3] Backport #2679 Update Certs for SSL #2684

Merged

[BUG] SecuritySSLReloadCertsActionTests is using outdated certificates #2675

Closed

cwperks mentioned this pull request Apr 18, 2023

Extensions config for JWT signing/encryption key #2671

Merged

3 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Update certs for SecuritySSLReloadCertsActionTests #2679

Update certs for SecuritySSLReloadCertsActionTests #2679

cwperks commented Apr 13, 2023

cwperks Apr 13, 2023

cwperks Apr 13, 2023

cwperks Apr 13, 2023

cwperks Apr 14, 2023 •

edited

Loading

cwperks Apr 14, 2023

codecov-commenter commented Apr 13, 2023 •

edited

Loading

cwperks commented Apr 14, 2023

cwperks commented Apr 14, 2023

stephen-crawford left a comment

opensearch-trigger-bot bot commented Apr 14, 2023

opensearch-trigger-bot bot commented Apr 14, 2023

Update certs for SecuritySSLReloadCertsActionTests #2679

Update certs for SecuritySSLReloadCertsActionTests #2679

Conversation

cwperks commented Apr 13, 2023

Description

Issues Resolved

Check List

cwperks Apr 13, 2023

Choose a reason for hiding this comment

cwperks Apr 13, 2023

Choose a reason for hiding this comment

cwperks Apr 13, 2023

Choose a reason for hiding this comment

cwperks Apr 14, 2023 • edited Loading

Choose a reason for hiding this comment

cwperks Apr 14, 2023

Choose a reason for hiding this comment

codecov-commenter commented Apr 13, 2023 • edited Loading

Codecov Report

cwperks commented Apr 14, 2023

cwperks commented Apr 14, 2023

stephen-crawford left a comment

Choose a reason for hiding this comment

opensearch-trigger-bot bot commented Apr 14, 2023

opensearch-trigger-bot bot commented Apr 14, 2023

cwperks Apr 14, 2023 •

edited

Loading

codecov-commenter commented Apr 13, 2023 •

edited

Loading