close
The Wayback Machine - https://web.archive.org/web/20201114121109/https://github.com/letsencrypt/boulder/pull/4384
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

deps: update github.com/zmap/zlint to latest. #4384

Merged
merged 2 commits into from Aug 1, 2019

Conversation

@cpu
Copy link
Contributor

@cpu cpu commented Aug 1, 2019

  • Update the github.com/zmap/zlint dependency to b126a9b, the tip of master at the time of writing. This captures a small fix to the ct_sct_policy_count_unsatisfied lint that ensures it isn't run for precertificates.
  • With that fix in place we no longer have to ignore the ct_sct_policy_count_unsatisfied lint in the config-next CA configs that enable preissuance linting.
Daniel added 2 commits Aug 1, 2019
Daniel
Update the `github.com/zmap/zlint` dependency to b126a9b. This captures
a small fix to the `ct_sct_policy_count_unsatisfied` lint that ensures
it isn't run for precertificates.
Daniel
With the latest `zlint` the `ct_sct_policy_count_unsatisfied` lint won't
flag precertificates as having an info-level lint result for missing
SCTs. With that fix in place we no longer have to ignore this lint in
the config-next CA configs that enable preissuance linting.
@cpu cpu requested a review from letsencrypt/boulder-developers as a code owner Aug 1, 2019
@cpu cpu self-assigned this Aug 1, 2019
@jsha
jsha approved these changes Aug 1, 2019
@rolandshoemaker rolandshoemaker merged commit 8b51845 into master Aug 1, 2019
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@rolandshoemaker rolandshoemaker deleted the cpu-zlint-bump-190801 branch Aug 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.