-
Notifications
You must be signed in to change notification settings - Fork 449
Add thumbprint_sha256 to code_signature.*
#2452
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
Conversation
|
Documentation changes preview: https://ecs_bk_2452.docs-preview.app.elstc.co/diff |
|
Warning It looks like this PR modifies one or more |
1 similar comment
|
Warning It looks like this PR modifies one or more |
|
@trisch-me as per our discussion in slack, and as I am only extending |
|
What do you think @mjwolf ? |
|
@ayfaouzi, I think that given this is one field, we don't need to go through the complete RFC process, but I would like to do some things from the RFC process. Could you:
|
45468be to
2efbad3
Compare
2efbad3 to
77bb0b0
Compare
|
Feedback addressed @jdu2600, thanks ! @joe-desimone can you please have a look ? |
thumbprint to code_signature.*thumbprint_sha256 to code_signature.*
jdu2600
left a comment
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.
LGTM.
Suggest that we include the pattern - even though it is a hint for now.
|
Can you please have a look again ? |


make test? Yesmakeand committed those changes? Yes