vanbroup / acme-auto-discovery

A mechanism for ACME clients to discover ACME server settings from a domain's CAA DNS record
https://vanbroup.github.io/acme-auto-discovery/draft-vanbrouwershaven-acme-auto-discovery.html
Other
5 stars 3 forks source link

Fix contradiction between definition and examples of priority #2

Closed aarongable closed 1 year ago

aarongable commented 1 year ago

Section 3.1.2:

the value "1" represents the highest priority, and subsequent values like "2", "3", and so on, indicate progressively lower priorities.

Section 3.2:

The CAA record with "ca2.example" has a higher priority value of 2, indicating it should be attempted first. The CAA record with "ca1.example" has a lower priority value of 1, indicating it should be attempted second.

This contradicts Section 3.1.2, which says that "2" is a lower priority than "1".

Section 3.2:

CAA records that do not explicitly specify a priority are automatically assigned the highest priority, which is 0.

This contradicts Section 3.1.2, which says that "1" is the highest priority.

I'd file a PR to fix this, but I'm not sure which direction you'd prefer to fix it in!

vanbroup commented 1 year ago

Thanks for spotting that, I would suggest that "1" represents the highest priority.

aarongable commented 1 year ago

Yeah, I think that the combination of:

probably leads to the most intuitive behavior

vanbroup commented 1 year ago

That works for me!