View Issue Details

IDProjectCategoryView StatusLast Update
0006933Talerexchangepublic2021-09-02 16:14
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
PrioritynormalSeverityminorReproducibilityhave not tried
Status closedResolutionfixed 
Product Versiongit (master) 
Target Version0.8Fixed in Version0.8 
Summary0006933: postgres_add_denomination_key reconnects inside transaction, potentially violating transaction atomicity
DescriptionIn the exchange, postgres_add_denomination_key does an auto-reconnect, but it is executed inside the add_keys transaction. Might not be severe, but doesn't look right.

This issue might also be present in other DB API functions.
TagsNo tags attached.

Required notifications

Address or linkRemarkAdded byNotifiedActions

Activities

Christian Grothoff

2021-07-22 13:50

manager   ~0018004

Fixed in 95f0fdf3..23e08471

Christian Grothoff

2021-09-02 16:14

manager   ~0018226

Fix committed to master branch.

Issue History

Date ModifiedUsernameFieldChange
2021-07-22 12:06Florian DoldNew Issue
2021-07-22 12:06Florian DoldStatusnew => assigned
2021-07-22 12:06Florian DoldAssigned To => Christian Grothoff
2021-07-22 13:50Christian GrothoffStatusassigned => informing
2021-07-22 13:50Christian GrothoffResolutionopen => fixed
2021-07-22 13:50Christian GrothoffFixed in Version => 0.9
2021-07-22 13:50Christian GrothoffNote Added: 0018004
2021-07-22 13:51Christian GrothoffProduct Version => git (master)
2021-07-22 13:51Christian GrothoffTarget Version => 0.9
2021-07-30 11:57Christian GrothoffFixed in Version0.9 => 0.8.1
2021-07-30 11:59Christian GrothoffTarget Version0.9 => 0.8.1
2021-07-30 12:01Christian GrothoffFixed in Version0.8.1 => 0.8
2021-07-30 12:02Christian GrothoffTarget Version0.8.1 => 0.8
2021-08-24 14:22Christian GrothoffStatusinforming => closed
2021-09-02 16:13Christian Grothoff
2021-09-02 16:14Christian GrothoffNote Added: 0018226