Skip to content
This repository has been archived by the owner on Oct 13, 2023. It is now read-only.

Transaction sender address is invalid #7

Open
hdevalence opened this issue Jun 7, 2023 · 1 comment
Open

Transaction sender address is invalid #7

hdevalence opened this issue Jun 7, 2023 · 1 comment
Assignees

Comments

@hdevalence
Copy link
Member

Using account group A on the web wallet and account group B on pcli, I sent funds from A to B, and viewed them in pcli:

~/c/p/penumbra2 ((v0.53.1)|✔) [101]$ cargo run --release --bin pcli -- view tx 0832ff091ab1ac228973b495460b8519bd2893fdd0d8aac41d87a88c7dd4d1a3
Finished release [optimized] target(s) in 0.33s
Running target/release/pcli view tx 0832ff091ab1ac228973b495460b8519bd2893fdd0d8aac41d87a88c7dd4d1a3
Scanning blocks from last sync height 77407 to latest height 77407
[0s] ██████████████████████████████████████████████████ 0/0 0/s ETA: 0s
Action Type Description
Spend [?] spent [?]
Output 100penumbra to [account 0]
Output [?] to [?]

Transaction Fee 0penumbra
Transaction Memo Sender penumbrav2t1rjzx23qh5fh6hh05gcaxljp8yl2gp5wh3vdvt9f40v4yjdzfhq3nu4dee8rtrtn065e4s5s46h0c0qdq8s3u5jgczur4yqwv36wrxkeu24tyu9ty3858qqfygn7vw26c2cgzcv
Transaction Memo Text
Transaction Expiration Height 0


But when attempting to send funds back to the sender address, I get an error:

~/c/p/penumbra2 ((v0.53.1)|✔) $ cargo run --release --bin pcli -- tx send 10pizza --to penumbrav2t1rjzx23qh5fh6hh05gcaxljp8yl2gp5wh3vdvt9f40v4yjdzfhq3nu4dee8rtrtn065e4s5s46h0c0qdq8s3u5jgczur4yqwv36wrxkeu24tyu9ty3858qqfygn7vw26c2cgzcv
Finished release [optimized] target(s) in 0.32s
Running target/release/pcli tx send 10pizza --to penumbrav2t1rjzx23qh5fh6hh05gcaxljp8yl2gp5wh3vdvt9f40v4yjdzfhq3nu4dee8rtrtn065e4s5s46h0c0qdq8s3u5jgczur4yqwv36wrxkeu24tyu9ty3858qqfygn7vw26c2cgzcv
Scanning blocks from last sync height 77419 to latest height 77419
[0s] ██████████████████████████████████████████████████ 0/0 0/s ETA: 0s
building transaction...
thread 'main' panicked at 'valid address: InvalidAddress', crates/core/transaction/src/plan/clue.rs:33:64
note: run with RUST_BACKTRACE=1 environment variable to display a backtrace

@Valentine1898
Copy link
Contributor

This is due to the fact that the default Memo was used in the construction of the transaction, in which the sender was filled with dummy address

@Valentine1898 Valentine1898 self-assigned this Jul 5, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants