Bug 1125030 - pulp bindings can't be used with the system certificate pack
Summary: pulp bindings can't be used with the system certificate pack
Alias: None
Product: Pulp
Classification: Retired
Component: z_other
Version: Master
Hardware: All
OS: Linux
Target Milestone: ---
: 2.4.0
Assignee: Randy Barlow
QA Contact: Preethi Thomas
Depends On:
TreeView+ depends on / blocked
Reported: 2014-07-30 22:34 UTC by Randy Barlow
Modified: 2014-08-09 06:55 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2014-08-09 06:55:29 UTC

Attachments (Terms of Use)

Description Randy Barlow 2014-07-30 22:34:00 UTC
We've used the m2crpto load_verify_locations() method to give m2crypto CA paths. Unfortunately with the capath parameter, the given directory must contain PEM certificates with only one certificate per file. This is fine for users who have generated their own CA who want to put that CA off in some random folder. However, on Red Hat based systems all installed CAs go together in one file (even custom generated ones).

load_verify_locations() accepts another parameter called cafile that can be a path to a file with multiple CAs in it. In order to work with system installed CAs, we will need to use that directive on Red Hat based systems because that's how Red Hat does things.

Unfortunately, many other operating systems (Ubuntu and Gentoo included) do the opposite - they install multiple files into a folder, one certificate per file.

I have no idea what m2crypto does if you use both directives, so I'm going to assume that we shouldn't do that. We could give the user access to both settings, but I think that would be confusing.

I propose that we do this instead:

1) We keep the ca_path setting, but we allow it to be a path to a directory or to a file,

2) When we are about to use the ca_path setting in m2crypto, we stat it to see which it is (directory or file). If a file, we use m2crypto's cafile directive. If a directory, we use m2crypto's capath directive. If it doesn't exist, we raise an exception (which is handy to catch typos anyway).

3) We document the junk out of this, so it's not confusing for users. We should provide good in-line docs in the files that use it as well as in our Sphinx docs.

4) We default it to /etc/pki/tls/certs/ca-bundle.crt, which contains Red Hat selected certificate authorities all in one file.

[0] http://www.heikkitoivonen.net/m2crypto/api/M2Crypto.SSL.Context%27.Context-class.html#load%5Fverify%5Flocations

Comment 1 Randy Barlow 2014-07-31 18:20:53 UTC

Comment 2 Randy Barlow 2014-08-06 18:23:40 UTC
Here are some steps to verify this bug:

1) Set all verify_ssl settings to True (consumer.conf, admin.conf, nodes.conf).

2) Leave ca_path to its default setting (/etc/pki/tls/certs/ca-bundle.crt)

3) Verify that pulp-admin <any command that talks to the server> gives you the ssl warning.

4) Generate a certificate authority certificate and key, and use them to sign Apache's certificates.

5) Copy the CA certificate (not the key!) to /etc/pki/ca-trust/source/anchors/

6) Run "update-ca-trust extract" as root (this will install the CA to the file at /etc/pki/tls/certs/ca-bundle.crt.)

7) Verify that pulp-admin is now willing to communicate with pulp server with no SSL warnings.

Comment 3 Preethi Thomas 2014-08-06 18:39:03 UTC
[root@cloud-qe-15 ~]# rpm -qa pulp-server
[root@cloud-qe-15 ~]# 

[root@cloud-qe-15 ~]# pulp-admin repo list

WARNING: The server's SSL certificate is untrusted!

The server's SSL certificate was not signed by a trusted authority. This could
be due to a man-in-the-middle attack, or it could be that the Pulp server needs
to have its certificate signed by a trusted authority. If you are willing to
accept the associated risks, you can set verify_ssl to False in the client
config's [server] section to disable this check.

[root@cloud-qe-15 ~]# 
[root@cloud-qe-15 ~]# cp newcerts/ca.crt /etc/pki/ca-trust/source/anchors/

[root@cloud-qe-15 ~]# update-ca-trust enable
[root@cloud-qe-15 ~]# update-ca-trust extract
[root@cloud-qe-15 ~]# 
[root@cloud-qe-15 ~]# 
[root@cloud-qe-15 ~]# pulp-admin repo list

Id:                  zoo
Display Name:        zoo
Description:         None
Content Unit Counts: 
  Erratum:          4
  Package Category: 1
  Package Group:    2
  Rpm:              32

[root@cloud-qe-15 ~]#

Comment 4 Randy Barlow 2014-08-09 06:55:29 UTC
This has been fixed in Pulp 2.4.0-1.

Note You need to log in before you can comment on or make changes to this bug.