Re: [PATCH] Accept IP addresses in server certificate SANs
Jacob Champion <pchampion@vmware.com>
From: Jacob Champion <pchampion@vmware.com>
To: "horikyota.ntt@gmail.com" <horikyota.ntt@gmail.com>
Cc: "pgsql-hackers@postgresql.org" <pgsql-hackers@postgresql.org>
Date: 2022-01-06T00:02:27Z
Lists: pgsql-hackers
Attachments
- v3-0001-Move-inet_net_pton-to-src-port.patch (text/x-patch) patch v3-0001
- v3-0002-libpq-allow-IP-address-SANs-in-server-certs.patch (text/x-patch) patch v3-0002
- v3-0003-squash-libpq-allow-IP-address-SANs-in-server-cert.patch (text/x-patch) patch v3-0003
On Mon, 2022-01-03 at 16:19 +0000, Jacob Champion wrote: > On Fri, 2021-12-17 at 15:40 +0900, Kyotaro Horiguchi wrote: > > > > + inet_net_pton_ipv4(const char *src, u_char *dst) > > (calls inet_net_pton_ipv4_internal(src, dst, true)) > > + inet_pton_ipv4(const char *src, u_char *dst) > > (calls inet_net_pton_ipv4_internal(src, dst, false)) > > Sounds good, I will make that change. Thanks for the feedback! v3 implements a pg_inet_pton(), but for IPv6 instead of IPv4 as presented above (since we only need inet_pton() for IPv6 in this case). It's split into a separate patch (0003) for ease of review. Thanks! --Jacob
Commits
-
libpq: Allow IP address SANs in server certificates
- c1932e542863 15.0 landed
-
Add SSL tests for IP addresses in certificates
- af9e18049550 15.0 landed
-
Make JSON path numeric literals more correct
- e26114c817b6 15.0 cited