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

setCode is showed as a Message signing #427

Closed
Tbaut opened this issue Oct 21, 2019 · 0 comments · Fixed by #428
Closed

setCode is showed as a Message signing #427

Tbaut opened this issue Oct 21, 2019 · 0 comments · Fixed by #428
Assignees

Comments

@Tbaut
Copy link
Contributor

Tbaut commented Oct 21, 2019

When scanning a multipart QR code for e.g system>setCode in apps, Signer shows to the user a hash to sign and doesn't give any hint regarding the method that we call.

We should instead show the usual Tx details view, but with a truncated payload e.g:

You will sign the transaction calling the method setCode with the following arguments:
code: "0x87EF...BEEF"

@Tbaut Tbaut added this to the v3.1 Secure enclave milestone Oct 21, 2019
@pmespresso pmespresso self-assigned this Oct 21, 2019
pmespresso added a commit that referenced this issue Oct 30, 2019
… Frames) (#428)

* fix: show prehash image along with hash

* fix: lint

* fix: display missed frames by index

* fix: remove from missed frames list once scanned

* fix: edge case for multiple  loops

* fix: multiple loops

* fix: off by one error

* fix: asyn cscanner state clear

* chore: bump @polkadot-js deps

* fix: make tests pass

* fix: qrcodehash

* fix: blake2s -> blake2b

* fix: signing hash

* fix: hash the correct thing ffs

* fix: stray print

* fix: show payload details along with hash

* fix: stray logs

* fix: minor grumble

* make android able to sign single part

* fix: ethereum sign msg

* fix: ethereum sign msg

* fix: undeifned

* fix: prehash optional
pmespresso added a commit that referenced this issue Nov 9, 2019
… Frames) (#428)

* fix: show prehash image along with hash

* fix: lint

* fix: display missed frames by index

* fix: remove from missed frames list once scanned

* fix: edge case for multiple  loops

* fix: multiple loops

* fix: off by one error

* fix: asyn cscanner state clear

* chore: bump @polkadot-js deps

* fix: make tests pass

* fix: qrcodehash

* fix: blake2s -> blake2b

* fix: signing hash

* fix: hash the correct thing ffs

* fix: stray print

* fix: show payload details along with hash

* fix: stray logs

* fix: minor grumble

* make android able to sign single part

* fix: ethereum sign msg

* fix: ethereum sign msg

* fix: undeifned

* fix: prehash optional
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants