Thursday, 2021-09-09

noonedeadpunkjrosser: one way for sure07:07
noonedeadpunkfor one way they also set ssl_ca (which is not really required but we use it throughout the roles)07:07
noonedeadpunkand public CA we generate with PKI role doesn't satisfy mysql client because of missing authority_cert_issuer07:08
noonedeadpunkBut it feels that client verify this filed only when CA is explicitly provided. And if not and taken from system trust store - it's fine07:09
noonedeadpunkhttps://mariadb.com/kb/en/secure-connections-overview/#server-certificate-verification doesn't mention it though....07:10
*** sshnaidm is now known as sshnaidm|off08:04
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-rabbitmq_server master: Fix PKI certificates regeneration  https://review.opendev.org/c/openstack/openstack-ansible-rabbitmq_server/+/80802108:28
opendevreviewDmitriy Rabotyagov proposed openstack/ansible-role-pki master: Ensure key and certificate when pki_regen_cert is defined  https://review.opendev.org/c/openstack/ansible-role-pki/+/80802208:30
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-haproxy_server master: Fix PKI regen behaviour  https://review.opendev.org/c/openstack/openstack-ansible-haproxy_server/+/80802308:36
opendevreviewDmitriy Rabotyagov proposed openstack/ansible-role-pki master: Ensure key and certificate regenerated when pki_regen_cert is defined  https://review.opendev.org/c/openstack/ansible-role-pki/+/80802208:44
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-galera_server master: Use ansible-role-pki to generate SSL certificates  https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/80771710:18
opendevreviewDmitriy Rabotyagov proposed openstack/ansible-role-pki master: Ensure key and certificate regenerated when pki_regen_cert is defined  https://review.opendev.org/c/openstack/ansible-role-pki/+/80802210:42
opendevreviewDmitriy Rabotyagov proposed openstack/ansible-role-pki master: Ensure key and certificate regenerated when pki_regen_cert is defined  https://review.opendev.org/c/openstack/ansible-role-pki/+/80802210:56
jrosserfeels like i missed something in the CSR which would make the cert be regenerated just by a new CSR12:49
jrosserfor the CA cert i think incrementing the serial number makes this happen automatically12:50
noonedeadpunkyeah, eventually you just need to do force for key and cert generation if key is set13:16
noonedeadpunkit works for me now in aio at least13:16
noonedeadpunkstill trying to understand how to generate  authority_key_identifier https://docs.ansible.com/ansible/latest/collections/community/crypto/openssl_csr_module.html#parameter-authority_key_identifier13:18
noonedeadpunkI tried just to set create_subject_key_identifier (hoped it can be fine to have that instead), but it's not13:19
jrosseri think there was complexity with that13:36
noonedeadpunkSo to get that you kind of need to have subjectPublicKey generated and only then alter already issued certificate with KeyIdentifier?13:41
noonedeadpunkif I understood https://stackoverflow.com/questions/22888574/how-to-get-authoritykeyidentifier-from-certificate correctly13:42
noonedeadpunkif yes, I'd say the best thing to do would be alter all roles to drop ca specification from connection string13:42
noonedeadpunkor we would need to create openssl.conf?13:43
noonedeadpunkhm...13:51
noonedeadpunkIt feels that it's present...  https://paste.opendev.org/show/809190/13:52
noonedeadpunkoh, damn13:56
noonedeadpunkwell, I guess whatever... let's maybe jsut repalce ssl-ca in connection string just to ssl?14:45
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-galera_server master: Use ansible-role-pki to generate SSL certificates  https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/80771714:47
noonedeadpunkI feel too stupid to understand what should be done here to solve the issue...14:48
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-galera_server master: Use ansible-role-pki to generate SSL certificates  https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/80771715:21
noonedeadpunkdamn... we can't define just ssl option https://github.com/PyMySQL/PyMySQL/blob/main/pymysql/connections.py#L266-L28415:28
noonedeadpunkbecause then pymysql expects it to be object....15:28
jrosserhrm sorry not being much help here15:28
mgariepyfun : https://jfrog.com/blog/critical-vulnerability-in-haproxy-cve-2021-40346-integer-overflow-enables-http-smuggling/15:29
noonedeadpunkso we must fix [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1131))15:29
noonedeadpunkdamn15:30
noonedeadpunkthankfully it doesn;t affect <2.015:31
noonedeadpunkjrosser: I just had some progress. So instead of placing intermediate as CA I placed root cert and it worked for mysql15:35
jrosserright15:36
jrosseri think thats expected?15:36
mgariepynoonedeadpunk, don't you have 20.04 clusters ?15:36
jrossernormally the intermediate is bundled with the server cert15:36
noonedeadpunkoh...15:36
jrosseri suspect that we have never used the galera ssl stuff before with cert + intermediate + root15:36
noonedeadpunkand in system trust store we put also root+intermediate right?15:39
mgariepythere is a workaround in this post: https://www.haproxy.com/blog/september-2021-duplicate-content-length-header-fixed/15:39
noonedeadpunkmgariepy: jsut upgraded first one to 20.0415:42
noonedeadpunkbut we have amphoras on 20.04 :(15:42
mgariepyneutron also have haproxy stuff irrc..15:43
mgariepyfor the metadata agent i thinkg15:43
noonedeadpunkyep15:44
mgariepynot sure if there is vulnerable config there tho.15:45
mgariepyamphora would probably be tho.15:45
noonedeadpunkI wonder if ubuntu has already patches this...15:46
noonedeadpunkit is according to https://linuxsecurity.com/advisories/ubuntu/ubuntu-5063-1-haproxy-vulnerabilities-09-14-1015:46
noonedeadpunkAnd I already have 2.0.13-2ubuntu0.315:47
noonedeadpunkhttps://ubuntu.com/security/notices/USN-5063-115:47
mgariepyho nice :D15:49
mgariepythe 2 ones for neutron are a pain also.15:51
mgariepydnsmask from aug 31st and the memory leak from this morning15:52
mgariepyer.. i meant .. dnsmasq** lol15:52
noonedeadpunkmgariepy: thanks for sharing actually!16:09
noonedeadpunkBecause I haven't noticed this one16:10
noonedeadpunkregarding ssl and mariadb - I guess I made some mistake in the role....16:30
noonedeadpunkbecause I was copying only intermediate as ca...16:30
*** frenzy_friday is now known as anbanerj|ruck17:21
mgariepynoonedeadpunk, no worries . i'm just lurking in #openstack-security.18:16
mgariepynoonedeadpunk, shall we bump the neutron sha a bit ? to fix both issue in patched release ?19:06

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!