-
Notifications
You must be signed in to change notification settings - Fork 112
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
Feat: [#1069] - added signature state id into the Digital signature #1070
Conversation
… to map the current signature state.
We discussed this a bit offline, and it looks like to avoid adding a new dictionary object, we could accomplish this by simply using the |
Thank you, that make sense. |
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.
This looks good to me
Signed-off-by: Pavel Jurka <pavel.jurka@sentinelone.com>
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!
description
Map the signature state to signature state id (+signature state as usual)
Related Issue:
#1069
Description of changes:
signature_state_id: optional
0 Unknown
1 Valid
2 Expired
3 Revoked
9 Other
signature_state: optional String as a value of signature_state_id, custom in a case of 99 as usual.