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

New cli argument allowing to specify receipt type to get after submission #212

Merged
merged 4 commits into from
Aug 23, 2024

Conversation

ivarprudnikov
Copy link
Member

@ivarprudnikov ivarprudnikov commented Aug 22, 2024

  • New argument --receipt-type with options raw (default), embedded, when calling submit action to allow the download of the embedded receipt in one step without going through embed-receipt command. This mitigates unwanted side effects when embed-receipt logic differs from the server logic; also, this will yield better user experience without the need to explain how to embed receipt in a separate step. It defaults to raw to avoid any breaking surprises for those who use the cli already.
  • New method get_last_embedded_receipt_from_cose which allows extracting the last receipt from a cose header. Used in receipt verification step when only signature with embedded receipt is provided.
  • Minor doc improvements

@ivarprudnikov ivarprudnikov force-pushed the feat/specify-receipt-type-when-submitting-cose branch from 5371b03 to 81ffcd6 Compare August 23, 2024 13:54

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
…ur of validate command
@ivarprudnikov ivarprudnikov enabled auto-merge (squash) August 23, 2024 16:09
@ivarprudnikov ivarprudnikov merged commit eddbef7 into main Aug 23, 2024
13 of 16 checks passed
@ivarprudnikov ivarprudnikov deleted the feat/specify-receipt-type-when-submitting-cose branch August 23, 2024 16:17
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

Successfully merging this pull request may close these issues.

None yet

2 participants