View Issue Details

IDProjectCategoryView StatusLast Update
0006737Talermerchant backendpublic2024-01-12 13:04
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 
Summary0006737: merchant should allow requests if authorization is not required, but "foreign" Authorization header is present
DescriptionThe merchant backend currently checks the syntax of the "Authorization" header before even checking if authorization is needed. The merchant *only* accepts bearer tokens in the Authorization header.

This complicates the deployment of the merchant backend where the reverse proxy handles authentication. The reverse proxy might not remove the authentication header before giving the request to the merchant backend.
TagsNo tags attached.

Required notifications

Address or linkRemarkAdded byNotifiedActions

Activities

Christian Grothoff

2021-02-04 17:35

manager   ~0017497

Should be fixed in e919d68..00c0256

Christian Grothoff

2021-09-02 16:23

manager   ~0018362

Fix committed to master branch.

Issue History

Date ModifiedUsernameFieldChange
2021-02-04 17:18Florian DoldNew Issue
2021-02-04 17:18Florian DoldStatusnew => assigned
2021-02-04 17:18Florian DoldAssigned To => Christian Grothoff
2021-02-04 17:35Christian GrothoffStatusassigned => informing
2021-02-04 17:35Christian GrothoffResolutionopen => fixed
2021-02-04 17:35Christian GrothoffFixed in Version => 0.9
2021-02-04 17:35Christian GrothoffNote Added: 0017497
2021-02-04 17:35Christian GrothoffFixed in Version0.9 => 0.8.1
2021-07-30 12:02Christian GrothoffFixed in Version0.8.1 => 0.8
2021-07-30 12:02Christian GrothoffTarget Version0.8.1 => 0.8
2021-08-24 14:23Christian GrothoffStatusinforming => closed
2021-09-02 16:22Christian Grothoff
2021-09-02 16:23Christian GrothoffNote Added: 0018362
2024-01-12 13:04Christian GrothoffCategorymerchant backend API (C) => mechant backend
2025-03-22 13:09Christian GrothoffCategorymechant backend => merchant backend