View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0007232 | Taler | merchant backoffice SPA | public | 2022-04-20 00:03 | 2022-11-04 20:52 |
Reporter | sebasjm | Assigned To | sebasjm | ||
Priority | low | Severity | tweak | Reproducibility | always |
Status | closed | Resolution | fixed | ||
Product Version | git (master) | ||||
Target Version | 0.9 | Fixed in Version | 0.9 | ||
Summary | 0007232: payto field should check bitcoin/ethereum addr checksum | ||||
Description | before the user submit the payto-URI in the bank account field in the new-instance form/ edit-instance form. right know there is a basic validation: bitcoin: ^(bc1|[13])[a-zA-HJ-NP-Z0-9]{25,39}$ eth: ^(0x)?[0-9a-f]{40}$ | ||||
Tags | No tags attached. | ||||
|
bitcoin is implemented, ethereum is still missing |
|
https://github.com/PaulRBerg/ethsum |
|
as pointed out here [1] checksum may also depend on `eip1191ChainId` and without this value we may create false positives. so I will leave the checksum check with the current implementation (valid HEX value of 40 chars) https://github.com/ethereumjs/ethereumjs-monorepo/blob/master/packages/util/src/account.ts#L138 copy of the comment: * Returns a checksummed address. * * If an eip1191ChainId is provided, the chainId will be included in the checksum calculation. This * has the effect of checksummed addresses for one chain having invalid checksums for others. * For more details see [EIP-1191](https://eips.ethereum.org/EIPS/eip-1191). * * WARNING: Checksums with and without the chainId will differ and the EIP-1191 checksum is not * backwards compatible to the original widely adopted checksum format standard introduced in * [EIP-55](https://eips.ethereum.org/EIPS/eip-55), so this will break in existing applications. * Usage of this EIP is therefore discouraged unless you have a very targeted use case. |
Date Modified | Username | Field | Change |
---|---|---|---|
2022-04-20 00:03 | sebasjm | New Issue | |
2022-04-20 00:03 | sebasjm | Status | new => assigned |
2022-04-20 00:03 | sebasjm | Assigned To | => sebasjm |
2022-04-21 13:30 | Christian Grothoff | Priority | normal => low |
2022-04-21 13:30 | Christian Grothoff | Severity | feature => tweak |
2022-07-17 16:34 | sebasjm | Note Added: 0018956 | |
2022-07-17 16:34 | Christian Grothoff | Note Added: 0018957 | |
2022-08-02 17:00 | sebasjm | Note Added: 0018982 | |
2022-08-02 17:00 | sebasjm | Status | assigned => resolved |
2022-08-02 17:00 | sebasjm | Resolution | open => fixed |
2022-10-20 10:47 | Christian Grothoff | Fixed in Version | => 0.9 |
2022-10-20 10:48 | Christian Grothoff | Target Version | git (master) => 0.9 |
2022-11-04 20:52 | Christian Grothoff | Status | resolved => closed |