-
Notifications
You must be signed in to change notification settings - Fork 686
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
fix: Verify V3 #5285
fix: Verify V3 #5285
Conversation
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.
💯
attestationId: attestation, | ||
hash: hashMessage(JSON.stringify(payload)), | ||
encryptedId, |
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.
@ganchoradkov isn't one of these (attestationId
, hash
, encryptedId
) redundant with the other?
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.
nope because
attestationId
= the jwthash
= the unecnrypted payload used for verify v1encryptedId
= hash of the encrypted message used for verify v2
encryptedId
is used for the check against jwt.payload.id
…^2.15.2 (#5871) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [@walletconnect/react-native-compat](https://github.com/walletconnect/walletconnect-monorepo) | [`^2.15.1` -> `^2.15.2`](https://renovatebot.com/diffs/npm/@walletconnect%2freact-native-compat/2.15.1/2.15.2) | [![age](https://developer.mend.io/api/mc/badges/age/npm/@walletconnect%2freact-native-compat/2.15.2?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/npm/@walletconnect%2freact-native-compat/2.15.2?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/npm/@walletconnect%2freact-native-compat/2.15.1/2.15.2?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/@walletconnect%2freact-native-compat/2.15.1/2.15.2?slim=true)](https://docs.renovatebot.com/merge-confidence/) | --- ### Release Notes <details> <summary>walletconnect/walletconnect-monorepo (@​walletconnect/react-native-compat)</summary> ### [`v2.15.2`](https://github.com/WalletConnect/walletconnect-monorepo/releases/tag/2.15.2) [Compare Source](https://github.com/walletconnect/walletconnect-monorepo/compare/2.15.1...2.15.2) ##### What's Changed - chore: prep for 2.15.1 release by [@​ganchoradkov](https://github.com/ganchoradkov) in [https://github.com/WalletConnect/walletconnect-monorepo/pull/5287](https://github.com/WalletConnect/walletconnect-monorepo/pull/5287) - feat: events sdk by [@​ganchoradkov](https://github.com/ganchoradkov) in [https://github.com/WalletConnect/walletconnect-monorepo/pull/5188](https://github.com/WalletConnect/walletconnect-monorepo/pull/5188) - chore: sets `trace` level logging on canary by [@​ganchoradkov](https://github.com/ganchoradkov) in [https://github.com/WalletConnect/walletconnect-monorepo/pull/5292](https://github.com/WalletConnect/walletconnect-monorepo/pull/5292) - fix: Verify V3 by [@​chris13524](https://github.com/chris13524) in [https://github.com/WalletConnect/walletconnect-monorepo/pull/5285](https://github.com/WalletConnect/walletconnect-monorepo/pull/5285) - Revert "chore: reverts verify v2" by [@​ganchoradkov](https://github.com/ganchoradkov) in [https://github.com/WalletConnect/walletconnect-monorepo/pull/5288](https://github.com/WalletConnect/walletconnect-monorepo/pull/5288) - fix(deps): update dependency elliptic to v6.5.7 \[security] by [@​renovate](https://github.com/renovate) in [https://github.com/WalletConnect/walletconnect-monorepo/pull/5275](https://github.com/WalletConnect/walletconnect-monorepo/pull/5275) - chore: doesn't init verify api in dev mode by [@​ganchoradkov](https://github.com/ganchoradkov) in [https://github.com/WalletConnect/walletconnect-monorepo/pull/5309](https://github.com/WalletConnect/walletconnect-monorepo/pull/5309) - fix: reconnection loop by [@​ganchoradkov](https://github.com/ganchoradkov) in [https://github.com/WalletConnect/walletconnect-monorepo/pull/5315](https://github.com/WalletConnect/walletconnect-monorepo/pull/5315) **Full Changelog**: WalletConnect/walletconnect-monorepo@2.15.1...2.15.2 </details> --- ### Configuration 📅 **Schedule**: Branch creation - "after 5pm,every weekend" in timezone America/Los_Angeles, Automerge - "after 5pm,every weekend" in timezone America/Los_Angeles. 🚦 **Automerge**: Enabled. ♻ **Rebasing**: Whenever PR is behind base branch, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR was generated by [Mend Renovate](https://mend.io/renovate/). View the [repository job log](https://developer.mend.io/github/valora-inc/wallet). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzOC41Ni4wIiwidXBkYXRlZEluVmVyIjoiMzguNTYuMCIsInRhcmdldEJyYW5jaCI6Im1haW4iLCJsYWJlbHMiOlsibnBtIiwicmVub3ZhdGUiXX0=--> --------- Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: valora-bot <valorabot@valoraapp.com>
…^2.15.2 (#5871) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [@walletconnect/react-native-compat](https://github.com/walletconnect/walletconnect-monorepo) | [`^2.15.1` -> `^2.15.2`](https://renovatebot.com/diffs/npm/@walletconnect%2freact-native-compat/2.15.1/2.15.2) | [![age](https://developer.mend.io/api/mc/badges/age/npm/@walletconnect%2freact-native-compat/2.15.2?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/npm/@walletconnect%2freact-native-compat/2.15.2?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/npm/@walletconnect%2freact-native-compat/2.15.1/2.15.2?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/@walletconnect%2freact-native-compat/2.15.1/2.15.2?slim=true)](https://docs.renovatebot.com/merge-confidence/) | --- ### Release Notes <details> <summary>walletconnect/walletconnect-monorepo (@​walletconnect/react-native-compat)</summary> ### [`v2.15.2`](https://github.com/WalletConnect/walletconnect-monorepo/releases/tag/2.15.2) [Compare Source](https://github.com/walletconnect/walletconnect-monorepo/compare/2.15.1...2.15.2) ##### What's Changed - chore: prep for 2.15.1 release by [@​ganchoradkov](https://github.com/ganchoradkov) in [https://github.com/WalletConnect/walletconnect-monorepo/pull/5287](https://github.com/WalletConnect/walletconnect-monorepo/pull/5287) - feat: events sdk by [@​ganchoradkov](https://github.com/ganchoradkov) in [https://github.com/WalletConnect/walletconnect-monorepo/pull/5188](https://github.com/WalletConnect/walletconnect-monorepo/pull/5188) - chore: sets `trace` level logging on canary by [@​ganchoradkov](https://github.com/ganchoradkov) in [https://github.com/WalletConnect/walletconnect-monorepo/pull/5292](https://github.com/WalletConnect/walletconnect-monorepo/pull/5292) - fix: Verify V3 by [@​chris13524](https://github.com/chris13524) in [https://github.com/WalletConnect/walletconnect-monorepo/pull/5285](https://github.com/WalletConnect/walletconnect-monorepo/pull/5285) - Revert "chore: reverts verify v2" by [@​ganchoradkov](https://github.com/ganchoradkov) in [https://github.com/WalletConnect/walletconnect-monorepo/pull/5288](https://github.com/WalletConnect/walletconnect-monorepo/pull/5288) - fix(deps): update dependency elliptic to v6.5.7 \[security] by [@​renovate](https://github.com/renovate) in [https://github.com/WalletConnect/walletconnect-monorepo/pull/5275](https://github.com/WalletConnect/walletconnect-monorepo/pull/5275) - chore: doesn't init verify api in dev mode by [@​ganchoradkov](https://github.com/ganchoradkov) in [https://github.com/WalletConnect/walletconnect-monorepo/pull/5309](https://github.com/WalletConnect/walletconnect-monorepo/pull/5309) - fix: reconnection loop by [@​ganchoradkov](https://github.com/ganchoradkov) in [https://github.com/WalletConnect/walletconnect-monorepo/pull/5315](https://github.com/WalletConnect/walletconnect-monorepo/pull/5315) **Full Changelog**: WalletConnect/walletconnect-monorepo@2.15.1...2.15.2 </details> --- ### Configuration 📅 **Schedule**: Branch creation - "after 5pm,every weekend" in timezone America/Los_Angeles, Automerge - "after 5pm,every weekend" in timezone America/Los_Angeles. 🚦 **Automerge**: Enabled. ♻ **Rebasing**: Whenever PR is behind base branch, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR was generated by [Mend Renovate](https://mend.io/renovate/). View the [repository job log](https://developer.mend.io/github/valora-inc/wallet). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzOC41Ni4wIiwidXBkYXRlZEluVmVyIjoiMzguNTYuMCIsInRhcmdldEJyYW5jaCI6Im1haW4iLCJsYWJlbHMiOlsibnBtIiwicmVub3ZhdGUiXX0=--> --------- Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: valora-bot <valorabot@valoraapp.com>
Description
Upgrades the Verify attestation style as per the v3 spec.
Remaining work:
Type of change
How has this been tested?
Existing automated tests
Checklist