Skip to content
This repository has been archived by the owner on Nov 24, 2021. It is now read-only.

Fixes #20054 - Add test if server's certificate is PEM encoded. #587

Merged
merged 1 commit into from Mar 14, 2018

Conversation

phayz
Copy link

@phayz phayz commented Feb 12, 2018

No description provided.

@theforeman-bot
Copy link

Issues: #20054

Copy link
Member

@chris1984 chris1984 left a comment

Choose a reason for hiding this comment

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

@phayz can you rebase and ill test this with a der cert I have.

@theforeman-bot
Copy link

@phayz, this pull request is currently not mergeable. Please rebase against the master branch and push again.

If you have a remote called 'upstream' that points to this repository, you can do this by running:

    $ git pull --rebase upstream master

This message was auto-generated by Foreman's prprocessor

@chris1984
Copy link
Member

now that is rebased, ill test with my der format certs

@chris1984
Copy link
Member

Works:

Checking server certificate's encoding: [FAIL]
The server certificate is in DER encoding, which is incompatible.
Run the following command to convert the certificate to PEM encoding,
then test it again.
# openssl x509 -in capsule.toledo.satellite.lab.eng.rdu2.redhat.com.der -outform pem -out capsule.toledo.satellite.lab.eng.rdu2.redhat.com.der.pem

When you run kcc.sh again, use file
capsule.toledo.satellite.lab.eng.rdu2.redhat.com.der.pem for the server certificate.

Thanks @phayz

@chris1984 chris1984 merged commit 097a7e2 into Katello:master Mar 14, 2018
@phayz phayz deleted the 20054 branch July 20, 2018 00:46
zjhuntin pushed a commit to zjhuntin/katello-installer that referenced this pull request Oct 30, 2018
…o#587)

(cherry picked from commit 5eeb68c3816cec9bccb7111128a4e6c9780b09ef)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
4 participants