-
Notifications
You must be signed in to change notification settings - Fork 109
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
AlgorithmIdentifier encoding (Section 7.1.3.1, CAB-Forum BR) #642
Conversation
This reverts commit 6c23670.
util: gtld_map autopull updates for 2021-10-21T07:25:20 UTC
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems reasonable to me, thank you!
v3/lints/cabf_br/lint_subject_public_key_info_improper_algorithm_object_identifier_encoding.go
Outdated
Show resolved
Hide resolved
v3/lints/cabf_br/lint_subject_public_key_info_improper_algorithm_object_identifier_encoding.go
Outdated
Show resolved
Hide resolved
...s/cabf_br/lint_subject_public_key_info_improper_algorithm_object_identifier_encoding_test.go
Outdated
Show resolved
Hide resolved
…hm_object_identifier_encoding.go Co-authored-by: Ryan Sleevi <ryan.sleevi@gmail.com>
I briefly wanted to ask whether this pull request is still considered for inclusion? Is there somethig to do on our side? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the heads up @christopher-henderson - Looks good to me.
This pull request adds a lint for Section 7.1.3.1 of CAB BR (introduced in v1.7.1 - https://cabforum.org/wp-content/uploads/CA-Browser-Forum-BR-1.7.1.pdf), in particular regarding the encoding of AlgorithmIdentifier for public keys. This lints is related to lint_mp_ecdsa_pub_key_encoding_correct.go. Please consider the lint for inclusion in zlint project.