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.

Sign up for GitHub

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

Jump to bottom

chore: Add nonce to Wallet Instance Attestation Request #49

Merged
merged 2 commits into from
Jul 8, 2023
Merged

chore: Add nonce to Wallet Instance Attestation Request #49

merged 2 commits into from
Jul 8, 2023

Conversation

grausof
Copy link
Collaborator

@grausof grausof commented Jul 4, 2023 •
edited
Loading

Based on what was established yesterday and with the addition of the nonce again it is necessary to add it also within the Wallet Instance Attestation Request otherwise the Wallet Provider will not be aware of the nonce!

The nonce can be used (following the necessary decisions #40) by the Wallet Solution for the use of Play Integrity Check

  • Ensure your files are written following RST specs (not MD!)
  • Italian version
  • English version
  • Example files
  • Ask for review

@peppelinux peppelinux marked this pull request as draft July 4, 2023 23:46
@peppelinux peppelinux marked this pull request as ready for review July 8, 2023 08:42
@peppelinux peppelinux merged commit fa9cf78 into versione-corrente Jul 8, 2023
@peppelinux peppelinux deleted the fix-wallet-instance-attestation branch July 12, 2023 16:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@peppelinux peppelinux Awaiting requested review from peppelinux

@fmarino-ipzs fmarino-ipzs Awaiting requested review from fmarino-ipzs

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@grausof @peppelinux