View Issue Details

IDProjectCategoryView StatusLast Update
0005991Talerexchangepublic2019-12-20 18:11
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
PrioritynormalSeverityfeatureReproducibilityN/A
Status closedResolutionfixed 
Product Versiongit (master) 
Target Version0.6Fixed in Version0.6 
Summary0005991: /wire response should include the master public key only once and at the top-level
DescriptionCurrently, each *wire account* details object contains a copy of the master public key. The wire fees don't contain any info about the master public key.

We should just include the master_pub *once* at the top-level of the response and omit it from the wire account details.
TagsNo tags attached.

Required notifications

Address or linkRemarkAdded byNotifiedActions

Activities

Christian Grothoff

2019-12-12 15:00

manager   ~0015147

As of 97810cc4..204358e2 (protocol version: 6:0:0) we now return the master_pub *also* in the top-level of the /wire response.

I did NOT remove it from the individual accounts (yet), for backwards-compatibility.

Florian Dold

2019-12-12 15:04

manager   ~0015148

At least for the wallets, removing the master_pub doesn't break anything. The wallet uses the master_pub from the /keys response to verify the signatures and only requires "url" and "master_sig" to be present.

Christian Grothoff

2019-12-12 15:07

manager   ~0015149

d5f4f6ff..b999a6b2 now marks the remaining change in the exchange logic, to be done once the wallet has been updated.

Issue History

Date ModifiedUsernameFieldChange
2019-12-12 14:40Florian DoldNew Issue
2019-12-12 14:40Florian DoldStatusnew => assigned
2019-12-12 14:40Florian DoldAssigned To => Christian Grothoff
2019-12-12 15:00Christian GrothoffNote Added: 0015147
2019-12-12 15:04Florian DoldNote Added: 0015148
2019-12-12 15:07Christian GrothoffNote Added: 0015149
2019-12-12 15:07Christian GrothoffAssigned ToChristian Grothoff => Florian Dold
2019-12-18 22:17Christian GrothoffStatusassigned => informing
2019-12-18 22:17Christian GrothoffResolutionopen => fixed
2019-12-18 22:17Christian GrothoffFixed in Version => 0.6
2019-12-18 22:17Christian GrothoffAssigned ToFlorian Dold => Christian Grothoff
2019-12-20 18:11Christian GrothoffStatusinforming => closed