diff options
author | Aaron Gable <aaron@aarongable.com> | 2021-01-14 04:12:48 +0300 |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-01-14 04:12:48 +0300 |
commit | 2fca48caaa8529432d003b0fdc880b673f6be1f5 (patch) | |
tree | 44212024e8c9ab62acc4ab10d96cd4effdd17807 /AUTHORS.md | |
parent | c0917a0302d10c9aca35f294b864499616b1af59 (diff) |
--preferred-chain: only match root name (#8596)
* --preferred-chain: only match root name
Currently, when certbot is given the `--preferred-chain='Some Name'`
flag, it iterates through all alternate chains offered by the ACME
server until it finds any certificate which has `'Some Name'` as its
Issuer Common Name. Unfortunately, this means that if the desired
alternate chain is a strict subset of any earlier chain (e.g. the
default chain is 'EE <-- Int <-- Root1 <-- Root2', but the desired
chain is 'EE <-- Int <-- Root1'), there is no name which can be
provided by the user which will allow the client to select the desired
chain.
This change makes it so that the `find_chain_with_issuer` logic only
cares about the Issuer Common Name found in the last certificate in
each chain. In the example above, the user would then be able to get
their desired chain by specifying `--preferred-chain='Root1'`: although
that name appears in the default chain, it does not appear in the
highest certificate of that chain.
This change is technically backwards-incompatible. However, the only
advice that has been given to users of certbot (and the only usecase
that we believe has existed so far) involved setting the flag to a
value that is the name of a root, not an intermediate, so we don't
expect any real-world configurations or use-cases to be broken.
Fixes #8577
* Update interfaces.py
Diffstat (limited to 'AUTHORS.md')
-rw-r--r-- | AUTHORS.md | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/AUTHORS.md b/AUTHORS.md index b00a90da3..cb60bfd87 100644 --- a/AUTHORS.md +++ b/AUTHORS.md @@ -1,6 +1,7 @@ Authors ======= +* [Aaron Gable](https://github.com/aarongable) * [Aaron Zirbes](https://github.com/aaronzirbes) * Aaron Zuehlke * Ada Lovelace |