Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

Add decryption to the UI (in the Signer) #5422

Merged
merged 3 commits into from
Apr 10, 2017
Merged

Conversation

ngotchac
Copy link
Contributor

@ngotchac ngotchac commented Apr 6, 2017

Using parity_decryptMessage RPC call would crash the Signer.

NB : The QR Signer code might not work with these methods (signing and decrypting)

@ngotchac ngotchac added A0-pleasereview 🤓 Pull request needs code review. M7-ui labels Apr 6, 2017
@@ -216,6 +216,8 @@ export function outSignerRequest (request) {
break;

case 'payload':
request[key].decrypt = outTransaction(request[key].decrypt);
request[key].sign = outTransaction(request[key].sign);
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sign or decrypt is not really a transaction, is it?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's true, it was out of laziness... Will update !

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated!

@tomusdrw tomusdrw added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Apr 9, 2017
@gavofyork gavofyork merged commit 0aaf236 into master Apr 10, 2017
@gavofyork gavofyork deleted the ng-add-decryption branch April 10, 2017 09:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A8-looksgood 🦄 Pull request is reviewed well.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants