-
Notifications
You must be signed in to change notification settings - Fork 350
Closed
Description
Current Behaviour:
The current client certificate code within gurl.py
in munkilib
uses this code, to validate whether a Certificate can be used to authenticate with the server.
munki/code/client/munkilib/gurl.py
Lines 714 to 717 in b720ed5
issuer_dict = dict(cert.native["tbs_certificate"]["issuer"]) | |
if issuer_dict in expected_issuer_dicts: | |
self.log("Found matching identity") | |
break |
This code currently doesn't walk up the Issuing Chain, so if a server only returns the accepted root certificates, Munki refuses to use a certificate to authenticate with the server, despite it being valid for authentication.
Expected Behaviour:
It's expected that when validating whether a certificate can be used for authentication, that the chain is walked to check for accepted certificates somewhere within the chain, and then use the certificate for authentication.
Metadata
Metadata
Assignees
Labels
No labels