View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0005668 | Taler | other | public | 2019-03-27 15:32 | 2019-04-20 13:01 |
| Reporter | Marcello Stanisci | 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 | always |
| Status | closed | Resolution | won't fix | ||
| Product Version | git (master) | ||||
| Target Version | 0.6 | Fixed in Version | 0.6 | ||
| Summary | 0005668: taler-auditor-exchange starts with NULL as config file | ||||
| Description | This command doesn't check if any config file was given on the command line, or does use some logic to pick a default config file (like say ~/.config/taler.conf) either. To be fixed. | ||||
| Tags | No tags attached. | ||||
| As a result, it uses the defaults from <prefix>/share/taler/config.d/ and often ends up trying to access non-existent databases. demo-blue@gv:~$ taler-deployment-keyup Mar 29 19:24:54-587764 taler-auditor-exchange-9941 ERROR Database connection failed: FATAL: database "taler" does not exist | |
| Eh, that's a feature. -c should be optional, and defaults to ~/.config/taler.conf, which is allowed to not exist (in which case the defaults take over, which ideally should be sufficient to run the tools to setup the rest. Naturally, we should probably have more automation: - taler-exchange-dbinit could try to setup the 'taler' database with PostGres - taler-exchange-keyup could require a --currency option and automatically configure denominations - taler-exchange-bank could be created to help the user configure the bank account details As a result, the user might not have to edit ~/.config/taler.conf by hand ever, just run the above commands in the right order. Regardless, making -c mandatory would be a regression IMO. | |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2019-03-27 15:32 | Marcello Stanisci | New Issue | |
| 2019-03-27 15:33 | Marcello Stanisci | Description Updated | |
| 2019-03-29 18:29 | Marcello Stanisci | Note Added: 0014240 | |
| 2019-04-02 17:36 | Christian Grothoff | Note Added: 0014257 | |
| 2019-04-20 13:01 | Christian Grothoff | Assigned To | => Christian Grothoff |
| 2019-04-20 13:01 | Christian Grothoff | Reproducibility | have not tried => always |
| 2019-04-20 13:01 | Christian Grothoff | Status | new => closed |
| 2019-04-20 13:01 | Christian Grothoff | Resolution | open => won't fix |
| 2019-04-20 13:01 | Christian Grothoff | Product Version | => git (master) |
| 2019-04-20 13:01 | Christian Grothoff | Fixed in Version | => 0.6 |
| 2019-04-20 13:01 | Christian Grothoff | Target Version | => 0.6 |