Skip to content

Add TLS-Certificate.Serial-Number-And-Issuer#5728

Open
minfrin wants to merge 1 commit into
FreeRADIUS:masterfrom
minfrin:snai
Open

Add TLS-Certificate.Serial-Number-And-Issuer#5728
minfrin wants to merge 1 commit into
FreeRADIUS:masterfrom
minfrin:snai

Conversation

@minfrin

@minfrin minfrin commented Jan 25, 2026

Copy link
Copy Markdown

The Serial Number and Issuer is a string format defined in RFC4523 called a CertificateExactAssertion that allows a unique instance of an X509 certificate to be identified independently of the contents of the certificate.

This interoperates with the SSL_CLIENT_CERT_RFC4523_CEA variable in Apache httpd's mod_ssl.

The Serial Number and Issuer is a string format defined in RFC4523 called a
CertificateExactAssertion that allows a unique instance of an X509 certificate
to be identified independently of the contents of the certificate.

This interoperates with the SSL_CLIENT_CERT_RFC4523_CEA variable in Apache
httpd's mod_ssl.
@arr2036

arr2036 commented Jan 25, 2026

Copy link
Copy Markdown
Member

@ndptech did you put the DER decoder behind a toggle yet?

@alandekok

Copy link
Copy Markdown
Member

@arr2036 The DER decoder is behind der_decode = yes in the TLS configuration.

@alandekok

Copy link
Copy Markdown
Member

This patch isn't needed any more. The DER code will now decode any arbitrary fields in a certificate.

We don't want to add more special-purpose code to v4.

@alandekok alandekok closed this May 24, 2026
@minfrin

minfrin commented May 24, 2026

Copy link
Copy Markdown
Author

This patch isn't needed any more. The DER code will now decode any arbitrary fields in a certificate.

We don't want to add more special-purpose code to v4.

I don't follow, the Serial-Number-And-Issuer is not a field in a certificate, but rather two fields in a certificate that are escape and encoded together in a special way described in RFC4523.

Right now today, neither the serial number nor the issuer is encoded in a way compatible with RFC4523, thus the need for a dedicated encoding.

@alandekok

Copy link
Copy Markdown
Member

I'll re-open this, but the unlang in v4 allows for fairly complex manipulation of attributes. It's possible that this kind of manipulation can be done with just normal unlang.

@alandekok alandekok reopened this May 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants