View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0006695 | Taler | documentation | public | 2021-01-13 23:47 | 2021-07-13 06:48 |
| Reporter | Florian Dold | Assigned To | ttn | ||
| Priority | high | Severity | text | Reproducibility | have not tried |
| Status | assigned | Resolution | open | ||
| Product Version | git (master) | ||||
| Summary | 0006695: review/edit the LibEuFin How-To | ||||
| Description | The How-To should be used by GLS to set up LibEuFin on their premises. Because LibEuFin provides a sandbox implementation of the EBICS protocol, it should be possible to follow all the steps in the How-To even without access to a real EBICS-enabled bank account. | ||||
| Tags | No tags attached. | ||||
| related to | 0006720 | resolved | MS | libeufin | nexus and sandbox should support getting the DB connection string from an environment variable |
| parent of | 0006729 | assigned | Florian Dold | Taler | Hackathon LibEuFin setup experience |
| Not all the children of this issue are yet resolved or closed. | |||||
|
|
I have pushed branch 'dev/ttn/nexus-howto-feedback': https://git.taler.net/docs.git/log/?h=dev/ttn/nexus-howto-feedback Please feel free to accept/reject the feedback as you see fit, but make sure you address/delete the comment/question (line that begins with ".. ???"). |
|
|
TTN: in my opinion you can merge the howto to the master branch. There might be divergences between the code samples and the real CLI because a few sub-commands got recently renamed. |
|
|
Also important: the DB connection string is now available via the environment, therefore the how-to should be adapted to this new style. |
|
|
OK, i've merged 'dev/ttn/nexus-howto-feedback' into 'master'. Thanks for taking a look. - Seems like the "Sandbox API" link does not render, even in its current formulation. Perhaps that's because it's in a "Note" block. I'll try to move it out and see what happens. - I'll work in the answer to the question (thanks!) in the next day or so. Generally, our docs don't show the result of each invoked command, but i'd like to slowly add that where it makes sense and is easy to do. This helps reassure the reader they're on the right track. - What is the "DB connection string" you mention? Are you talking about env vars like 'PGHOST' and 'PGDATA', or something else entirely? |
|
|
The DB connection string a path to the database. That path is a string in the Java's JDBC format. The following two examples show a SQLite and PostgreSQL path. 1. jdbc:sqlite:/tmp/libeufintestdb 2. jdbc:postgresql://localhost:$port/libeufintestdb?user=$username&password=$password |
|
|
I understand now. Thanks. When you say "is available in the environment" what does that mean? Is there a specific variable that has these strings (or one of them) as its value? |
|
|
Exactly. There are two environment variables: LIBEUFIN_NEXUS_DB_CONNECTION and LIBEUFIN_SANDBOX_DB_CONNECTION. Those two variables get read (respectively) by Nexus and Sandbox in order to obtain a DB connection string. And at any given time, each variable is supposed to hold only one connection string. Does that answer? |
|
|
Yes, thanks. Last question: When both ways are specified (command-line and env var), which one takes precedence? I imagine command-line, since that's how most programs behave, but i would like your confirmation. |
|
|
Oops, i see from https://bugs.gnunet.org/view.php?id=6720 that the command-line is no longer supported. Sorry for the noise! |
|
|
Florian, MS: I think we can close this issue. WDYT? |
|
|
Libeufin changed in the meanwhile: only the Debian packaged version has a default database. The How-To should now say that LIBEUFIN_{SANDBOX,NEXUS}_DB_CONNECTION should be set before trying any operation. But I'm still not 100% convinced of this solution: what do you/Florian think? Shall LibEuFin provide a default for a database? One reason of the removal was that the user should _know_ where the database is, and not dig through the logs to discover where it is located. |
|
|
It's fine for me if the DB doesn't have a default. However, it would be great if the error message you get when you don't specify the database includes an example sqlite db connection string, just to be user friendly. |
|
|
Fixed. I'd say that TTN can now adapt the document (and close this). |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2021-01-13 23:47 | Florian Dold | New Issue | |
| 2021-01-13 23:47 | Florian Dold | Status | new => assigned |
| 2021-01-13 23:47 | Florian Dold | Assigned To | => ttn |
| 2021-01-21 04:07 | ttn | Note Added: 0017382 | |
| 2021-01-26 19:05 | MS | Note Added: 0017395 | |
| 2021-01-26 19:06 | MS | Note Edited: 0017395 | View Revisions |
| 2021-01-26 19:07 | MS | Note Edited: 0017395 | View Revisions |
| 2021-01-26 19:27 | MS | Note Added: 0017399 | |
| 2021-01-27 02:51 | ttn | Note Added: 0017404 | |
| 2021-01-27 12:24 | MS | Note Added: 0017407 | |
| 2021-01-27 12:25 | MS | Note Edited: 0017407 | View Revisions |
| 2021-01-27 13:24 | ttn | Note Added: 0017411 | |
| 2021-01-27 14:08 | MS | Note Added: 0017413 | |
| 2021-01-29 13:59 | ttn | Note Added: 0017444 | |
| 2021-01-29 15:22 | ttn | Note Added: 0017445 | |
| 2021-01-29 15:23 | ttn | Relationship added | related to 0006720 |
| 2021-01-29 19:40 | ttn | Note Added: 0017448 | |
| 2021-01-29 21:12 | MS | Note Added: 0017450 | |
| 2021-01-29 21:20 | Florian Dold | Note Added: 0017451 | |
| 2021-01-29 22:05 | MS | Note Added: 0017452 | |
| 2021-01-30 14:57 | ttn | Relationship added | parent of 0006729 |
| 2021-01-30 23:55 | Christian Grothoff | Severity | minor => text |
| 2021-06-02 09:57 | Christian Grothoff | Assigned To | ttn => Christian Grothoff |
| 2021-07-13 06:48 | Christian Grothoff | Assigned To | Christian Grothoff => ttn |
