Rework SIA handling to be less incorrect
authortb <tb@openbsd.org>
Mon, 3 Jun 2024 12:58:39 +0000 (12:58 +0000)
committertb <tb@openbsd.org>
Mon, 3 Jun 2024 12:58:39 +0000 (12:58 +0000)
commitc5305b1da563242eb69f257b4d535147d939611b
tree30ea28a61734c5b1ebad8f0816eb9f27060607f1
parenta09e95841b3ee2acbb8dccab10902d61ea0f0031
Rework SIA handling to be less incorrect

Currently we would only accept rsync URIs in caRepository and rpkiManifest.
This is wrong. The specification requires at least one rsync URI to be
resent, but allows for other URI types, ordered by descending CA preference.
With this diff we will still only respect the first rsync address, but we
will no longer reject certificates listing other URIs. Make the warning
conditional on verbose mode.

Also make it clearer that sbgp_sia() (which is poorly named) is for CA
certs while x509_get_sia() is for EE certs.

input/ok claudio
usr.sbin/rpki-client/cert.c
usr.sbin/rpki-client/x509.c