zecwallet icon indicating copy to clipboard operation
zecwallet copied to clipboard

Include Reply Address bug

Open leto opened this issue 5 years ago • 1 comments

If a user selects an address, then writes a memo and clicks "Include Reply Address" THEN goes and changes the address where funds are coming from, the included reply address is no longer correct. I found this pretty easy to do in the GUI and I almost sent a message with the wrong reply address.

For the sake of the user, it may be best to use a regex to remove any existing Reply To: headers in the memo and add the current Reply To: header, whenever that button is hit.

This is a very small edge case bug, non-critical. Replies still go to the correct wallet and user, just not the right address. This could cause some accounting headaches and confusion, though.

leto avatar Jun 03 '19 12:06 leto

Ah, yes. I hadn't though of this. I can make it always pick up the "from" field.

adityapk00 avatar Jun 03 '19 21:06 adityapk00