View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0008891 | Taler | libeufin-bank | public | 2024-06-03 16:56 | 2024-07-26 00:11 |
| Reporter | sebasjm | Assigned To | Antoine A | ||
| Priority | normal | Severity | major | Reproducibility | always |
| Status | closed | Resolution | won't fix | ||
| Product Version | 0.12 | ||||
| Target Version | 0.12 | Fixed in Version | 0.12 | ||
| Summary | 0008891: invalid payto create | ||||
| Description | i can create a bank account with underscore in the username if the bank is x-taler-bank it will create my payto will include _ `_` is not in [the spec](https://www.rfc-editor.org/rfc/rfc8905.html#name-syntax-of-a-payto-uri) either it should remove the _ or it should not allow to create username with _ | ||||
| Tags | No tags attached. | ||||
|
|
Removing the _ could create collisions, so I would be in favor of making the username format more restrictive. Currently, the regex is [a-zA-Z0-9\-\._~]+. If we want to align with payto's specifications we should remove support for _ and ~: [a-zA-Z0-9\-\.]+. I will ask christian about it |
|
|
To fully comply with the spec I think it should be [a-zA-Z][a-zA-Z0-9\-\.]* |
|
|
The payto uri currently generated does follow the specification, I'm closing waiting for more details when it fails again with another component. |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2024-06-03 16:56 | sebasjm | New Issue | |
| 2024-06-03 16:56 | sebasjm | Status | new => assigned |
| 2024-06-03 16:56 | sebasjm | Assigned To | => Antoine A |
| 2024-06-06 04:13 | Antoine A | Note Added: 0022521 | |
| 2024-06-06 04:24 | Antoine A | Note Added: 0022522 | |
| 2024-06-10 09:08 | Antoine A | Status | assigned => resolved |
| 2024-06-10 09:08 | Antoine A | Resolution | open => won't fix |
| 2024-06-10 09:08 | Antoine A | Note Added: 0022562 | |
| 2024-07-26 00:11 | Christian Grothoff | Fixed in Version | => 0.12 |
| 2024-07-26 00:11 | Christian Grothoff | Status | resolved => closed |
