Feature #12272
closed
Support for multiple certificates in ca.crt for oVirt
Description
In app/models/compute_resources/foreman/model/ovirt.rb ca_cert_store() function stores retrieved ca.crt in OpenSSL::X509::Store object.
The problem is, OpenSSL::X509::Certificate.new(cert) only takes into account the last certificate in cert.
If cert contains more than one certificate (which is quite common on production systems these days), only last certificate in the chain will be added to the store, and SSL verification in oVirt will not work.
This blocks the Foreman usage with RHEV-M.
The code below fixed issue for me, though I'm not a real Ruby programmer and am sure there's better way to do this.
Main idea is certificates should be split and added to the OpenSSL::X509::Store one by one.
def ca_cert_store cert
return if cert.blank?
s = OpenSSL::X509::Store.new
splitcert = ""
cert_arr = []
i = 0
cert.each_line do |line|
splitcert += line
if line =~ /-----END [^\-]+-----/
cert_arr << splitcert
splitcert = ""
end
end
cert_arr.each do |c|
s.add_cert(OpenSSL::X509::Certificate.new(c.to_s))
end
s
end
I can send a pull request if the above approach is fine.
Updated by Lukas Zapletal about 9 years ago
AFAIK Foreman supports chain of CA certificates in this field. From our web UI helper text: "Optionally provide a CA, or a correctly ordered CA chain. If left blank, a self-signed CA will be populated automatically by the server during the first request". Make sure the order is correct. I have tested this and OpenSSL seems to work. Tested on RHEL, what platform do you use?
Updated by Christophe Roux almost 9 years ago
I am having the exact same issue (on RHEL7m Satellite 6.1.6) and the proposed workaround is working.
It really seems that the command OpenSSL::X509::Certificate.new(cert) is only taking the last cert.
require 'openssl'
require 'socket'
cert=File.read("./rhevm.pem")
s=OpenSSL::X509::Certificate.new(cert)
cert_store=OpenSSL::X509::Store.new.add_cert(s)
ssl_context = OpenSSL::SSL::SSLContext.new
ssl_context.cert_store = cert_store
ssl_context.set_params(verify_mode: OpenSSL::SSL::VERIFY_PEER)
tcp_socket = TCPSocket.open 'rhevm.example.com', 443
ssl_socket = OpenSSL::SSL::SSLSocket.new tcp_socket, ssl_context
ssl_socket.connect
is returning,
SSL_connect returned=1 errno=0 state=SSLv3 read server certificate B: certificate verify failed (OpenSSL::SSL::SSLError)but the following using add_file method (which the doc clearly says support multiple certificates)
require 'openssl'
require 'socket'
cert_store=OpenSSL::X509::Store.new.add_file("./rhevm.pem")
ssl_context = OpenSSL::SSL::SSLContext.new
ssl_context.cert_store = cert_store
ssl_context.set_params(verify_mode: OpenSSL::SSL::VERIFY_PEER)
tcp_socket = TCPSocket.open 'rhevm.example.com', 443
ssl_socket = OpenSSL::SSL::SSLSocket.new tcp_socket, ssl_context
ssl_socket.connect
Works fine
Updated by Ori Rabin over 7 years ago
- Status changed from New to Assigned
- Assignee set to Ori Rabin
Updated by The Foreman Bot over 7 years ago
- Status changed from Assigned to Ready For Testing
- Pull request https://github.com/theforeman/foreman/pull/4411 added
Updated by Ori Rabin over 7 years ago
- Status changed from Ready For Testing to Closed
- % Done changed from 0 to 100
Applied in changeset 4c3516219692e729204a1cd4a12283c61c8e3b62.
Updated by Ohad Levy over 7 years ago
- Translation missing: en.field_release set to 209
Updated by