View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0004858 | Taler | exchange | public | 2017-01-13 01:21 | 2017-06-06 14:18 |
Reporter | Florian Dold | Assigned To | Florian Dold | ||
Priority | normal | Severity | minor | Reproducibility | have not tried |
Status | closed | Resolution | fixed | ||
Product Version | 0.3 | ||||
Target Version | 0.3 | Fixed in Version | 0.3 | ||
Summary | 0004858: exchange should initialize database automatically, just like the merchant does | ||||
Description | For two reasons: 1. restores consistency with taler-merchant-{httpd,dbinit} 2. makes deployment scripts easier, since we don't have to start the db separately, wait for it and initialize the DB before we can start the exchange | ||||
Tags | No tags attached. | ||||
|
We should not _always_ do the dbinit automatically as it is costly and would make us slow when fuzzing or restarting from a crash. Also, there is the issue of "IF NOT EXISTS" not being supported (yet) for indices. And as usual, running less logic is better. However, I don't see a reason why we should not have an *option* to do dbinit within the httpd. Note that uniformity with the merchant's httpd is a non-goal for me, as the exchange better be run by admins that know exactly what they do, and for them trying to create a table that already exists is a sign of a MAJOR screw-up (in fact, the permission to create tables should have been revoked prior to going into production!), while for merchants my expectation is _WAY_ lower and the usability concerns are _WAY_ higher. |
|
Fixed with https://git.taler.net/exchange.git/commit/?id=0bfb5b25091a1cbdd9911ae124ae6da35cd2b916 |
Date Modified | Username | Field | Change |
---|---|---|---|
2017-01-13 01:21 | Florian Dold | New Issue | |
2017-01-13 01:21 | Florian Dold | Status | new => assigned |
2017-01-13 01:21 | Florian Dold | Assigned To | => Christian Grothoff |
2017-01-13 11:42 | Christian Grothoff | Note Added: 0011627 | |
2017-01-13 14:47 | Florian Dold | Assigned To | Christian Grothoff => Florian Dold |
2017-01-13 14:47 | Florian Dold | Status | assigned => resolved |
2017-01-13 14:47 | Florian Dold | Resolution | open => fixed |
2017-01-13 14:47 | Florian Dold | Note Added: 0011630 | |
2017-01-16 10:19 | Christian Grothoff | Fixed in Version | => 0.3 |
2017-06-06 14:18 | Christian Grothoff | Status | resolved => closed |