Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Define "CT Qualified" in terms of Days, not Months #1

Open
sleevi opened this issue Apr 24, 2017 · 2 comments
Open

Define "CT Qualified" in terms of Days, not Months #1

sleevi opened this issue Apr 24, 2017 · 2 comments

Comments

@sleevi
Copy link
Contributor

sleevi commented Apr 24, 2017

Currently, the number of SCTs related to the Lifetime of Certificate is defined in months. However, this leaves it ambiguous as to how to calculate it.

Several possible interpretations have been arrived at:

  • Treating every month as 31 days, thus 15 months is 465 days
  • Treating 15 months as the maximum legitimate period (366 days, followed by a 3 months sequence of July/Aug/Sept at 31 + 31 + 30), or 458 days
  • A complex scheme of rounding, such that: (notAfter.year - notBefore.year) * 12 + (notAfter.month - notBefore.month), and then subtracting one if notAfter.day < notBefore.day.

Expressing this in days is seen as least ambiguous.

For Chrome, this is https://bugs.chromium.org/p/chromium/issues/detail?id=713362

@sleevi
Copy link
Contributor Author

sleevi commented Apr 25, 2017

devonobrien added a commit that referenced this issue Aug 4, 2017
@devonobrien
Copy link
Collaborator

In the spirit of simplifying implementation and reducing ambiguity, we're changing the CT Qualified table to reflect certificate lifetimes in terms of days, not months. The second of the above three interpretations was chosen for month-to-day conversion:

  • Treating 15 months as the maximum legitimate period (366 days, followed by a 3 months sequence of July/Aug/Sept at 31 + 31 + 30), or 458 days

The other ranges are listed in the following comment in the Chromium bug:
https://bugs.chromium.org/p/chromium/issues/detail?id=713362#c3

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

No branches or pull requests

2 participants