View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0006780 | Taler | merchant backend | public | 2021-03-02 20:56 | 2021-09-02 16:23 |
| Reporter | Florian Dold | Assigned To | Christian Grothoff | Resolution deadline | |
| Implementer | Reviewer | ||||
| Release signed by | Deployed by | ||||
| Verifier | Communicator | ||||
| Data protection authority notification required | No | ||||
| CSIRT notification required | No | ||||
| Estimated effort (hours) | Actual effort (hours) | 0 | |||
| Fast-track | No | ||||
| Work package | none | ||||
| Priority | normal | Severity | minor | Reproducibility | have not tried |
| Status | closed | Resolution | fixed | ||
| Product Version | git (master) | ||||
| Target Version | 0.8 | Fixed in Version | 0.8 | ||
| Summary | 0006780: merchant backend interprets some instance URLs incorrectly | ||||
| Description | Can be reproduced with this test: $ taler-wallet-cli testing run-integrationtests merchant-instances-urls These currently go wrong: http://localhost:8083/instances/foobar/config => This one should return 404 if instance "foobar" doesn't exist, but it returns 200 http://localhost:8083/private/instances/myinst/orders => This one should also return 404, as "instances/myinst/orders" is not a private endpoint of the default instance. Furthermore, in this case it would be ambiguous if this refers to the public or private "orders" endpoint. | ||||
| Tags | No tags attached. | ||||
| Additionally reported by sebasjm: http://localhost:8083/instances/foo/private/instances => This one should 404, as "foo" is not the default instances and thus not allowed to manage instances. But it currently seems to return 200. | |
| 3923209..1ca25d6 should fix these. | |
| Note that /instances/default/private/instances now intentionally (!) also 404s. /private/instances is ONLY available at the top-level. | |
| One of the test cases is still failing: http://localhost:8083/instances/default/private/instances => The test case wants this to be 200, but the merchant backend gives 404. Of course it's debatable which one is correct here. But I would think the following base URLs should be *equivalent* in every aspect: http://localhost:8083/instances/default/ http://localhost:8083/ right?! | |
| Oop, didn't read your previous comment. Seems fine. | |
| Fix committed to master branch. | |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2021-03-02 20:56 | Florian Dold | New Issue | |
| 2021-03-02 20:56 | Florian Dold | Status | new => assigned |
| 2021-03-02 20:56 | Florian Dold | Assigned To | => Christian Grothoff |
| 2021-03-03 20:29 | Florian Dold | Note Added: 0017588 | |
| 2021-03-04 11:10 | Christian Grothoff | Note Added: 0017595 | |
| 2021-03-04 11:11 | Christian Grothoff | Note Added: 0017596 | |
| 2021-03-04 11:12 | Christian Grothoff | Status | assigned => informing |
| 2021-03-04 11:12 | Christian Grothoff | Resolution | open => fixed |
| 2021-03-04 11:12 | Christian Grothoff | Fixed in Version | => 0.9 |
| 2021-03-04 11:12 | Christian Grothoff | Product Version | => git (master) |
| 2021-03-04 11:12 | Christian Grothoff | Target Version | => 0.9 |
| 2021-03-04 11:54 | Florian Dold | Status | informing => assigned |
| 2021-03-04 11:54 | Florian Dold | Note Added: 0017597 | |
| 2021-03-04 11:55 | Florian Dold | Note Added: 0017598 | |
| 2021-03-04 11:55 | Florian Dold | Status | assigned => informing |
| 2021-07-30 11:57 | Christian Grothoff | Fixed in Version | 0.9 => 0.8.1 |
| 2021-07-30 11:59 | Christian Grothoff | Target Version | 0.9 => 0.8.1 |
| 2021-07-30 12:02 | Christian Grothoff | Fixed in Version | 0.8.1 => 0.8 |
| 2021-07-30 12:02 | Christian Grothoff | Target Version | 0.8.1 => 0.8 |
| 2021-08-24 14:23 | Christian Grothoff | Status | informing => closed |
| 2021-09-02 16:22 | Christian Grothoff | ||
| 2021-09-02 16:23 | Christian Grothoff | Note Added: 0018357 | |
| 2025-03-22 13:09 | Christian Grothoff | Category | mechant backend => merchant backend |