<div dir="ltr">Hi Evgeniy,<div>what you've proposed is all right, although it adds some overhead for certificate provisioning.</div><div>In fact, to do it right we should probably define REST API for provisioning certificates.</div><div>I'm rather for simplified approach, consisting of Shell / Puppet scripts for certificate upload/management.</div><div><br></div><div>Regards,</div><div><br></div><div>A.</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Aug 21, 2015 at 12:20 PM, Evgeniy L <span dir="ltr"><<a href="mailto:eli@mirantis.com" target="_blank">eli@mirantis.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi Stanislaw,<div><br></div><div>I agree that user's certificates mustn't be saved in Nailgun database, in cluster attributes,</div><div>in this case it can be seen in all the logs, which is terrible security problem,</div><div>and we already have a place where we keep auto-generated certificates and</div><div>ssh-keys, and those are copied to specific nodes by Astute.</div><div><br></div><div>So UI should send the file to specific URL, Nginx should be configured to send auth</div><div>request to backend, after request is authorised, Nginx should save the file into predefined</div><div>directory, the same which we use for autogenerated certificates, in this case such</div><div>tool as OSTF can take certificates from a single place.</div><div><br></div><div>Thanks,</div></div><div class="gmail_extra"><br><div class="gmail_quote"><span class="">On Fri, Aug 21, 2015 at 12:10 PM, Stanislaw Bogatkin <span dir="ltr"><<a href="mailto:sbogatkin@mirantis.com" target="_blank">sbogatkin@mirantis.com</a>></span> wrote:<br></span><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5"><div dir="ltr"><span style="font-size:12.8000001907349px">Hi folks.</span><div style="font-size:12.8000001907349px"><br></div><div style="font-size:12.8000001907349px">Today I want to discuss the way we save SSL keys for Fuel environments. As you maybe know we have 2 ways to get a key:</div><div style="font-size:12.8000001907349px">a. Generate it by Fuel (self-signed certificate will be created in this case). In this case we will generate private key, csr and crt in a pre-deployment hook on master node and then copy keypair to the nodes which needed it.</div><div style="font-size:12.8000001907349px"><br></div><div style="font-size:12.8000001907349px">b. Get a pre-generated keypair from user. In this case user should create keypair by himself and then upload it through Fuel UI settings tab. In this case keypair will be saved in nailgun database and then will serialized into astute.yaml on cluster nodes, pulled from it by puppet and saved into a file.</div><div style="font-size:12.8000001907349px"><br></div><div style="font-size:12.8000001907349px">Second way has some flaws:</div><div style="font-size:12.8000001907349px">1. We already have some keys for nodes and we store them on master node. Store keys in different places is bad, cause:</div><div style="font-size:12.8000001907349px">1.1. User experience - user should remember that in some cases keys will be store in FS and in some other cases - in DB.</div><div style="font-size:12.8000001907349px">1.2. It brings problems for implementation in other different places - for example, we need to get certificate for properly run OSTF tests and now we should implement two different ways to deliver that certificate to OSTF container. The same for fuel-cli - we should somehow get certificate from DB and place it in FS to use it.</div><div style="font-size:12.8000001907349px">2. astute.yaml is similar for all nodes. Not all of nodes needs to have private key, but now we cannot control this.</div><div style="font-size:12.8000001907349px">3. If keypair data serializes into astute.yaml it means than that data automatically will be fetched when diagnostic snapshot will created. So in some cases in can lead to security vulnerability, or we will must to write another crutch to cut it out of diagnostic snapshot.</div><div style="font-size:12.8000001907349px"><br></div><div style="font-size:12.8000001907349px"><br></div><div style="font-size:12.8000001907349px">So I propose to get rid of saving keypair in nailgun database and implement a way to always saving it to local FS on master node. We need to implement next items:</div><div style="font-size:12.8000001907349px"><br></div><div style="font-size:12.8000001907349px">- Change UI logic that saving keypair into DB to logic that will save it to local FS</div><div style="font-size:12.8000001907349px">- Implement according fixes in fuel-library</div></div>
<br></div></div><span class="">__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
<br></span></blockquote></div><br></div>
<br>__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr"><div style="color:rgb(136,136,136);font-size:12.8000001907349px">Adam Heczko</div><div style="color:rgb(136,136,136);font-size:12.8000001907349px">Security Engineer @ Mirantis Inc.</div></div></div>
</div>