View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0006778 | Taler | merchant backend | public | 2021-03-02 19:53 | 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 | 0006778: merchant-lib and testing support for authentication unclear | ||||
| Description | Currently the merchant lib does not have any explicit support for setting and changing authentication headers. There is GNUNET_CURL_append_header, but that does not seem to allow *removing* a header. Do we want to manage authentication through this, or should the merchant lib functions take some extra context parameter for authentication? The same concern applies the the merchant's testing lib, which currently can't easily set/remove the authentication header. | ||||
| Tags | No tags attached. | ||||
| related to | 0006779 | closed | Christian Grothoff | merchant-lib does not support changing instance authentication |
| The preferred method is: /* initialize HTTP client */ ctx = GNUNET_CURL_init (&GNUNET_CURL_gnunet_scheduler_reschedule, &rc); rc = GNUNET_CURL_gnunet_rc_create (ctx); /* setup authorization */ GNUNET_assert (GNUNET_OK == GNUNET_CURL_append_header (ctx, authorization)); Note that if you must change the headers (i.e. different Authorization), you should create a fresh ctx/rc each time. | |
| 4087bfb..657d5ba adds a command to test 0006779 (posting to /private/auth). | |
| da25913e..0453ef45 adds a command (to taler_testing_lib) to add/update/remove an authorization header from the CURL context. | |
| 657d5ba..08d7955 adds tests and minor bugfixes. Resolving this issue. | |
| Fix committed to master branch. | |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2021-03-02 19:53 | Florian Dold | New Issue | |
| 2021-03-02 19:53 | Florian Dold | Status | new => assigned |
| 2021-03-02 19:53 | Florian Dold | Assigned To | => Christian Grothoff |
| 2021-03-02 22:40 | Christian Grothoff | Note Added: 0017582 | |
| 2021-03-02 22:40 | Christian Grothoff | Assigned To | Christian Grothoff => Florian Dold |
| 2021-04-16 21:10 | Christian Grothoff | Assigned To | Florian Dold => Christian Grothoff |
| 2021-04-17 17:38 | Christian Grothoff | Note Added: 0017754 | |
| 2021-04-17 17:38 | Christian Grothoff | Relationship added | related to 0006779 |
| 2021-04-17 18:08 | Christian Grothoff | Note Added: 0017756 | |
| 2021-04-17 19:49 | Christian Grothoff | Note Added: 0017757 | |
| 2021-04-17 19:49 | Christian Grothoff | Status | assigned => informing |
| 2021-04-17 19:49 | Christian Grothoff | Resolution | open => fixed |
| 2021-04-17 19:49 | Christian Grothoff | Fixed in Version | => 0.9 |
| 2021-04-17 19:49 | Christian Grothoff | Target Version | => 0.9 |
| 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:01 | 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: 0018350 | |
| 2025-03-22 13:09 | Christian Grothoff | Category | mechant backend => merchant backend |