View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0007006 | GNUnet | postgres library | public | 2021-08-28 19:22 | 2022-02-26 23:10 |
| Reporter | Florian Dold | Assigned To | Christian Grothoff | ||
| Priority | normal | Severity | minor | Reproducibility | have not tried |
| Status | closed | Resolution | fixed | ||
| Product Version | Git master | ||||
| Target Version | 0.16.0 | Fixed in Version | 0.16.0 | ||
| Summary | 0007006: merchant dbinit does not properly report errors | ||||
| Description | It just tells me "3". $ sudo -u taler-merchant-httpd taler-merchant-dbinit -c /etc/taler/taler.conf 2021-08-28T17:17:37.295109+0000 taler-merchant-dbinit-418734 WARNING Could not run PSQL on file /usr/share/taler//sql/merchant/merchant-0000.sql: 3 2021-08-28T17:17:37.295186+0000 taler-merchant-dbinit-418734 ERROR Failed to run SQL logic to setup database versioning logic 2021-08-28T17:17:37.295238+0000 taler-merchant-dbinit-418734 ERROR Failed to initialize tables | ||||
| Tags | No tags attached. | ||||
|
|
That's because it doesn't have more: it forks the "psql" binary, and the only thing we have is the psql binary exit code. |
|
|
Ah. But then why don't we just let psql inherit stderr, so we get a decent error message? And currently the error message *looks* broken. Is the "3" a line number, an exit status or ...? At least we should indicate in the log text that this is the exit status of psql. |
|
|
Fixed error message text and inherited stderr as of 7a4c1fb72..2994fa434 |
|
|
(Note that this is a GNUnet change.) |
|
|
0.16.0 released |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2021-08-28 19:22 | Florian Dold | New Issue | |
| 2021-08-28 19:22 | Florian Dold | Status | new => assigned |
| 2021-08-28 19:22 | Florian Dold | Assigned To | => Christian Grothoff |
| 2021-08-28 20:23 | Christian Grothoff | Note Added: 0018139 | |
| 2021-08-28 20:23 | Christian Grothoff | Assigned To | Christian Grothoff => Florian Dold |
| 2021-08-28 20:23 | Christian Grothoff | Status | assigned => feedback |
| 2021-08-28 21:00 | Florian Dold | Note Added: 0018140 | |
| 2021-08-28 21:54 | Christian Grothoff | Note Added: 0018141 | |
| 2021-08-28 21:54 | Christian Grothoff | Note Added: 0018142 | |
| 2021-08-28 21:54 | Christian Grothoff | Status | feedback => resolved |
| 2021-08-28 21:54 | Christian Grothoff | Resolution | open => fixed |
| 2021-08-28 21:54 | Christian Grothoff | Fixed in Version | => 0.8.5 |
| 2021-08-28 21:55 | Christian Grothoff | Project | Taler => GNUnet |
| 2021-08-28 21:55 | Christian Grothoff | Category | mechant backend => General |
| 2021-08-28 21:55 | Christian Grothoff | Assigned To | Florian Dold => Christian Grothoff |
| 2021-08-28 21:55 | Christian Grothoff | Category | General => postgres library |
| 2021-08-28 21:55 | Christian Grothoff | Product Version | => Git master |
| 2021-08-28 21:55 | Christian Grothoff | Fixed in Version | 0.8.5 => 0.16.0 |
| 2021-08-28 21:55 | Christian Grothoff | Target Version | => 0.16.0 |
| 2021-10-10 20:14 | schanzen | Fixed in Version | 0.16.0 => 0.15.4 |
| 2021-10-10 20:14 | schanzen | Target Version | 0.16.0 => 0.15.4 |
| 2021-12-31 09:26 | schanzen | Target Version | 0.15.4 => 0.16.0 |
| 2022-02-22 23:06 | schanzen | Fixed in Version | => 0.16.0 |
| 2022-02-26 23:10 | schanzen | Note Added: 0018730 | |
| 2022-02-26 23:10 | schanzen | Status | resolved => closed |
