View Issue Details

IDProjectCategoryView StatusLast Update
0007851Talerwallet (WebExtension)public2023-09-23 13:09
Reportersebasjm Assigned Tosebasjm 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
PrioritynormalSeveritymajorReproducibilitysometimes
Status closedResolutionwon't fix 
Product Versiongit (master) 
Fixed in Version0.9.3 
Summary0007851: we need a way to test that a new query does not break CORS requirement
DescriptionI'm happy to take this but after we agree how we can do it and IF we can do it.

The current situation:
When wallet core queries /config of demobank it is adding the header

```
Access-Control-Request-Headers: 0
```

in the OPTION request.

Chrome is not doing the OPTION request.
Firefox does it, and fails because the response of the server does not match with the request requirement.

easy to fix, but hard to find.

What I want with this issue is that we found a way to prevent this issue to happen, maybe with unit test or with a clear message error.

Things to consider:

 * CORS request failure are opaque: usually seen as "request failed"
 * Depends on the platform: is it possible to simulate in node?
 * The error may also depend on the server response
TagsNo tags attached.
Attached Files
Selection_124.png (52,450 bytes)   
Selection_124.png (52,450 bytes)   
Selection_123.png (121,855 bytes)   
Selection_123.png (121,855 bytes)   

Required notifications

Address or linkRemarkAdded byNotifiedActions

Activities

sebasjm

2023-05-31 14:40

developer   ~0020210

More on this:
I was using Firefox 113.0.2 (64-bit) Mozilla firefox snap for ubuntu canonical-002 - 1.0

After uninstalling and installing again the Addon, the problem persist.
After uninstalling, restarting firefox and installing again the Addon, the problem persist.

Look for firefox older version in https://ftp.mozilla.org/pub/firefox/releases
Trying with version 104.0.2, the problem is not present.
Trying with the same version (113.0.2) but another installation, the problem is not present.

At this point, I assume that my current firefox instance had a configuration problem.

Removing the addon, installing from web store. Problem not present, it is an older version but the relevant code that make the http request is the same. (v 0.9.3.10 from Mar 31, 2023)

After removing the removing the version from websotre and installing from source again, the problem is not preset !!!!
And source compilation in the middle.

Very strange.

sebasjm

2023-06-05 14:12

developer   ~0020263

I saw this again, Chrome 114.0.5735.90
The request was not using the network yet CORS problem were reported by the browser.
After cleaning the browser cache the problem was gone.

Florian Dold

2023-08-02 16:07

manager   ~0020392

@Sebastian: Not sure what to do about this, given the fundamental browser limitations.

Do you think any action is required or can we close this?

sebasjm

2023-08-02 16:23

developer   ~0020393

we agree that we don't have enough information in these cases to show better information to the user

Issue History

Date ModifiedUsernameFieldChange
2023-05-31 13:33sebasjmNew Issue
2023-05-31 13:33sebasjmStatusnew => assigned
2023-05-31 13:33sebasjmAssigned To => Florian Dold
2023-05-31 13:33sebasjmFile Added: Selection_124.png
2023-05-31 13:33sebasjmFile Added: Selection_123.png
2023-05-31 13:34sebasjmDescription Updated
2023-05-31 13:35sebasjmCategorywallet-core => wallet (WebExtension)
2023-05-31 14:40sebasjmNote Added: 0020210
2023-06-05 14:12sebasjmNote Added: 0020263
2023-08-02 16:07Florian DoldAssigned ToFlorian Dold => sebasjm
2023-08-02 16:07Florian DoldStatusassigned => feedback
2023-08-02 16:07Florian DoldNote Added: 0020392
2023-08-02 16:23sebasjmStatusfeedback => informing
2023-08-02 16:23sebasjmResolutionopen => won't fix
2023-08-02 16:23sebasjmNote Added: 0020393
2023-09-23 13:07Christian GrothoffFixed in Version => 0.9.3
2023-09-23 13:09Christian GrothoffStatusinforming => closed