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

CEW + Ledger Wallet (Ledger Nano S) - "invalid sender" #134

Closed
cypheron opened this issue May 3, 2018 · 1 comment
Closed

CEW + Ledger Wallet (Ledger Nano S) - "invalid sender" #134

cypheron opened this issue May 3, 2018 · 1 comment

Comments

@cypheron
Copy link

cypheron commented May 3, 2018 •
edited

clo_ledger_bug
Sending CLO via Ledger Nano S is not possible due to an error message "invalid sender" upon confirming the sending of the transaction.

Steps to reproduce:

  1. Connect and unlock ledger, select "Ethereum" app on Ledger
  2. Open CEW in Chrome, Select Network "CLO (Callisto.network"
  3. "Send Ether & Tokens" > "Ledger Wallet" > "Connect to Ledger Wallet"
  4. Select address with CLO on it, set receiver address, amount and gas price
  5. "Generate transaction" > Confirm on ledger > "Send transaction"
  6. On pop-up dialog, press "Yes, I'm sure. Make transaction".

Upon confirming the dialog an error message appears with the title "invalid sender". See screenshot for details.

Note: The confirmation dialog on Ledger Nano S does not mention CLO but ETH.

This is a different error than in #117!
@cypheron cypheron changed the title Ledger Nano S: "invalid sender" Bug: CEW + Ledger Wallet (Ledger Nano S) - "invalid sender" May 3, 2018
@cypheron cypheron changed the title Bug: CEW + Ledger Wallet (Ledger Nano S) - "invalid sender" CEW + Ledger Wallet (Ledger Nano S) - "invalid sender" May 3, 2018
j-chimienti pushed a commit to j-chimienti/etherwallet that referenced this issue May 4, 2018
update ledger gen / send tx
f1e6423 
fixes:

change HD derivation path - EthereumCommonwealth#128

invalid sender / send CLO - EthereumCommonwealth#134, EthereumCommonwealth#117

changes:

update hd path on change
remove eip155 support ledger w/ clo
update location hash w/ view msg
define wallet props in service
update err message send msg
@j-chimienti j-chimienti mentioned this issue May 4, 2018
Merged
@cypheron
Copy link
Author

cypheron commented May 5, 2018

Fixed with #136.

@cypheron cypheron closed this as completed May 5, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@cypheron