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 |