View Issue Details

IDProjectCategoryView StatusLast Update
0004335Talerobsolete componentpublic2016-03-31 19:59
ReporterChristian Grothoff Assigned ToMarcello Stanisci 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
PrioritylowSeveritytextReproducibilityN/A
Status closedResolutionfixed 
Platformi7OSDebian GNU/LinuxOS Versionsqueeze
Product Version0.0 
Target Version0.0Fixed in Version0.0 
Summary0004335: bank uses (hardcoded?) database 'talertest' which is odd
DescriptionFirst of all, hardcoding the database name everywhere in the code is probably a bad idea. OTOH, the user probably shouldn't be prompted to supply it all the time, so I think having a 'configure' option (--databasename=) would probably be ideal.

Second, the current default is also awkward, as 'test' is still used by the running system. I would suggest 'talerbank' as the default.

Ideally, we'd have such a more sane default, and configure _and_ runtime options to override the default.
TagsNo tags attached.

Required notifications

Address or linkRemarkAdded byNotifiedActions

Activities

Marcello Stanisci

2016-03-23 13:09

reporter   ~0010321

Add file bank_project/settings.py.in with @tokentosubstitute@ wherever needed,
like in db's name field, so that the configure process will yield the final
settings.py as the user wishes

Marcello Stanisci

2016-03-31 13:44

reporter   ~0010342

How to add your --customoption to configure:

http://stackoverflow.com/questions/27416003/create-custom-configure-command-line-arguments

Marcello Stanisci

2016-03-31 14:11

reporter   ~0010343

fixed in d2685de..1d47c57

served with '--with-dbname' option

Issue History

Date ModifiedUsernameFieldChange
2016-03-23 08:49Christian GrothoffNew Issue
2016-03-23 08:49Christian GrothoffStatusnew => assigned
2016-03-23 08:49Christian GrothoffAssigned To => Marcello Stanisci
2016-03-23 08:49Christian GrothoffPrioritynormal => low
2016-03-23 08:49Christian GrothoffTarget Version0.0 => 0.1
2016-03-23 13:09Marcello StanisciNote Added: 0010321
2016-03-31 13:44Marcello StanisciNote Added: 0010342
2016-03-31 14:11Marcello StanisciNote Added: 0010343
2016-03-31 14:11Marcello StanisciStatusassigned => informing
2016-03-31 14:11Marcello StanisciResolutionopen => fixed
2016-03-31 16:30Christian GrothoffStatusinforming => closed
2016-03-31 19:59Christian GrothoffFixed in Version => 0.0
2016-03-31 19:59Christian GrothoffTarget Version0.1 => 0.0
2022-08-23 18:26Christian GrothoffCategorybank (demonstrator) => py bank (demonstrator, obsolete)
2023-12-03 00:23Christian GrothoffCategorypy bank (demonstrator, obsolete) => obsolete componet
2023-12-11 19:08Florian DoldCategoryobsolete componet => obsolete component