View Issue Details

IDProjectCategoryView StatusLast Update
0006129Talerspecificationpublic2021-09-02 16:23
Reportergrote 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
PriorityhighSeveritymajorReproducibilityhave not tried
Status closedResolutionfixed 
Product Versiongit (master) 
Target Version0.7.1Fixed in Version0.7.1 
Summary0006129: Merchant API not idempotent, does not behave as documented
DescriptionThis relates to the refund API: https://docs.taler.net/core/api-merchant.html#giving-refunds

I've been testing this and when sending a `RefundRequest` POST request,
I get back:

{
        "sig": "5TJNEY2W9DB...",
        "contract_terms": {
            ...
        },
        "taler_refund_uri":
"taler:\/\/refund\/backend.test.taler.net\/-\/-\/2020..."
}

which is fine, but not exactly a `MerchantRefundResponse` as documented.

Also, when doing the same request again, I get a 409 error. Which is not
so nice if for some reason, the merchant went back and needs to let the
customer scan again.

So I tried an undocumented GET request and there I get something that
looks almost like a MerchantRefundResponse, but there's no
refund_redirect_url. So if I missed the URL the first time, there seems
to be no way to get it back.
TagsNo tags attached.

Required notifications

Address or linkRemarkAdded byNotifiedActions

Activities

Christian Grothoff

2020-04-07 16:53

manager   ~0015562

Fixed in 7cfd98c..e781a77: The API should now be idempotent AND follow the updated and simplified specification.

Christian Grothoff

2021-09-02 16:23

manager   ~0018382

Fix committed to master branch.

Issue History

Date ModifiedUsernameFieldChange
2020-03-17 11:37groteNew Issue
2020-03-17 11:37groteStatusnew => assigned
2020-03-17 11:37groteAssigned To => Christian Grothoff
2020-04-07 14:12Christian GrothoffPrioritynormal => high
2020-04-07 14:12Christian GrothoffSeverityminor => major
2020-04-07 14:12Christian GrothoffProduct Version => git (master)
2020-04-07 14:12Christian GrothoffTarget Version => 0.7.1
2020-04-07 16:53Christian GrothoffStatusassigned => informing
2020-04-07 16:53Christian GrothoffResolutionopen => fixed
2020-04-07 16:53Christian GrothoffFixed in Version => 0.7.1
2020-04-07 16:53Christian GrothoffNote Added: 0015562
2021-08-24 14:23Christian GrothoffStatusinforming => closed
2021-09-02 16:22Christian Grothoff
2021-09-02 16:23Christian GrothoffNote Added: 0018382
2024-01-12 13:02Christian GrothoffCategorymerchant backend API (HTTP specification) => specification