View Issue Details

IDProjectCategoryView StatusLast Update
0006034Talerexchangepublic2021-01-17 14:54
ReporterFlorian Dold Assigned ToChristian Grothoff Resolution deadline
ImplementerReviewer 
Release signed byDeployed by 
VerifierCommunicator 
Data protection authority notification requiredNo  
CSIRT notification requiredNo  
Estimated effort (hours)Actual effort (hours)0 
Fast-trackNo
Work packagenone
PrioritylowSeverityfeatureReproducibilityhave not tried
Status closedResolutionwon't fix 
Product Versiongit (master) 
Summary0006034: Exchange should have separate configuration file
DescriptionAs discussed in a private e-mail, this has a bunch of advantages:

* It is easier to validate a configuration, as we can warn about unused sections and options in some sanity checking tool

* Having one file per component makes it easier to generate more complex setups for testing (i.e. two merchants, two auditors, maybe even two exchanges with different currencies)

* We have had configuration section "clashes" in the past, where the merchant parsed a section intended for the exchange, due to the same prefix. Separate files avoid this issue.
TagsNo tags attached.

Required notifications

Address or linkRemarkAdded byNotifiedActions

Activities

Christian Grothoff

2020-12-20 19:29

manager   ~0017221

The current exchange has substantially fewer options, a LOT has moved into the database.
Many of the remaining options (like credentials for how to authenticate against the bank) are indeed very tool-specific and in high-security deployments should be in tool-specific files and not some 'global' configuration.

So overall, my feeling is that this bug was always questionable, but now it is even more so.

Florian: Can we just wontfix it?

Issue History

Date ModifiedUsernameFieldChange
2020-01-07 18:17Florian DoldNew Issue
2020-01-07 18:17Florian DoldStatusnew => assigned
2020-01-07 18:17Florian DoldAssigned To => Christian Grothoff
2020-01-16 23:33Christian GrothoffTarget Version0.7.0 => 0.9
2020-01-16 23:33Christian GrothoffAssigned ToChristian Grothoff =>
2020-01-16 23:33Christian GrothoffStatusassigned => acknowledged
2020-01-17 20:40Christian GrothoffPrioritynormal => low
2020-03-28 21:20Christian GrothoffSeverityminor => feature
2020-07-24 10:07Christian GrothoffTarget Version0.9 => 0.9.1
2020-12-20 19:29Christian GrothoffNote Added: 0017221
2021-01-17 14:54Christian GrothoffAssigned To => Christian Grothoff
2021-01-17 14:54Christian GrothoffStatusacknowledged => closed
2021-01-17 14:54Christian GrothoffResolutionopen => won't fix
2021-01-17 14:54Christian GrothoffFixed in Version => 0.8.1
2021-01-17 14:54Christian GrothoffTarget Version0.9.1 => 0.8.1