View Issue Details

IDProjectCategoryView StatusLast Update
0007535Talerwallet (WebExtension)public2023-02-21 15:39
Reporterdavidak 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
PrioritynormalSeverityminorReproducibilityalways
Status closedResolutionfixed 
Product Version0.9 
Target Version0.9.2Fixed in Version0.9.2 
Summary0007535: camera image is mirrored, unable to scan qr code
Description.
Steps To Reproduce1. withdraw money in bank
2. scan qr code in browser wallet extension with webcam
3. qr code not detected, because the image is mirrored
Additional InformationWallet Core
0.9.0 673e7a417d043cd6b0644d2a7a52d1fdbc5345ee
Web Extension
0.9.0.35 673e7a417d043cd6b0644d2a7a52d1fdbc5345ee

webcam is https://www.amazon.de/gp/product/B085ZKNJF3/ref=ppx_yo_dt_b_asin_title_o08_s01?ie=UTF8&th=1
it is not mirrored in OBS or other websites like https://de.webcamtests.com/

Google Chrome 108.0.5359.124
on NixOS 22.11.1137.dac57a4eccf
with Pantheon desktop
TagsNo tags attached.
Attached Files

Required notifications

Address or linkRemarkAdded byNotifiedActions

Activities

davidak

2023-01-30 13:18

reporter   ~0019715

It's actually the whole media player that is mirrored including the (hidden by default) controls.

sebasjm

2023-01-30 15:19

developer   ~0019716

Looks like the library is doing a css transformation to flip the camera based on MediaStream api

https://github.com/nimiq/qr-scanner/blob/master/src/qr-scanner.ts#L366
https://github.com/nimiq/qr-scanner/blob/abcfe1bce2703721408d8ce7ebde94a359998506/src/qr-scanner.ts#L877
https://github.com/nimiq/qr-scanner/blob/abcfe1bce2703721408d8ce7ebde94a359998506/src/qr-scanner.ts#L935

Also, the worker implementation is using URL.createObjectURL which is not present in every browser.
I'm going to move to another implementation, probably https://github.com/cozmo/jsQR

davidak

2023-01-30 16:28

reporter   ~0019717

That works.

sebasjm

2023-01-31 13:22

developer   ~0019718

fixed at 9d9a88af010ac39f026299ebccea3e1164e5242e

moved to a lower level implementation, tested on firefox and chrome
also implemented reading qrcode from file

Issue History

Date ModifiedUsernameFieldChange
2022-12-25 18:23davidakNew Issue
2022-12-25 18:23davidakStatusnew => assigned
2022-12-25 18:23davidakAssigned To => Florian Dold
2022-12-25 18:23davidakFile Added: Screenshot from 2022-12-25 19-16-32.png
2023-01-08 09:17Christian GrothoffAssigned ToFlorian Dold => sebasjm
2023-01-08 09:17Christian GrothoffTarget Version => 0.9.2
2023-01-30 13:18davidakNote Added: 0019715
2023-01-30 13:18davidakFile Added: Screenshot from 2023-01-29 10-33-28.png
2023-01-30 15:19sebasjmNote Added: 0019716
2023-01-30 16:28davidakNote Added: 0019717
2023-01-30 16:28davidakFile Added: Screenshot from 2023-01-30 17-27-37.png
2023-01-31 13:22sebasjmStatusassigned => informing
2023-01-31 13:22sebasjmResolutionopen => fixed
2023-01-31 13:22sebasjmFixed in Version => git (master)
2023-01-31 13:22sebasjmNote Added: 0019718
2023-02-17 15:33Christian GrothoffFixed in Versiongit (master) => 0.9.2
2023-02-21 15:39Christian GrothoffStatusinforming => closed
2023-04-13 18:37Florian DoldCategorywallet (WebExtensions) => wallet (WebExtension)