pdf417 not scanning

[joleeee]

Describe the bug

This might be an upstream issue. See also gruhn/vue-qrcode-reader#459

Basically, barqode will detect and scan PDF417 codes, but the output is emptystring "".

I tested again today with a real boarding pass and got nothing, while using a random website scanner actually decodes the data properly.

from https://barqode.sveco.dev/docs/demos/full-demo:

from random website (working)

Reproduction

1. https://barqode.sveco.dev/docs/demos/full-demo
2. try to scan a PDF417 code, ex.

Logs

System Info

System:
    OS: macOS 14.5
    CPU: (8) arm64 Apple M2
    Memory: 354.34 MB / 24.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 20.16.0 - ~/.volta/tools/image/node/20.16.0/bin/node
    Yarn: 4.0.0-rc.51 - ~/.volta/tools/image/yarn/4.0.0-rc.51/bin/yarn
    npm: 10.8.1 - ~/.volta/tools/image/node/20.16.0/bin/npm
    pnpm: 8.8.0 - ~/Library/pnpm/pnpm
    bun: 1.0.5 - ~/.volta/bin/bun
  Browsers:
    Chrome: 133.0.6943.53
    Safari: 17.5

Severity

blocking all usage of Barqode

[ollema]

interesting! will investigate

[joleeee]

it works on iOS

i tried safari on mac, and that would not even produce ""

[ollema]

could you try https://zxing-wasm-demo.deno.dev/

zxing-wasm is the polyfill that is used when there is no native support. we have not updated that in a while so maybe it works upstream.

I’m thinking that whether or not this polyfill is active could explain when we run into these issues.

also could you include the console logs from barqode? it should indicate if the polyfill is used IIRC.

[joleeee]

Yes, https://zxing-wasm-demo.deno.dev/ works perfectly.

I think this line from the console is relevant:

[barqode] will use native BarcodeDetector.

Whole log:

Removing unpermitted intrinsics
barqode-stream.C4fzdaJK.js:1 [barqode] starting camera with constraints:  {"facingMode":"environment"}
barqode-stream.C4fzdaJK.js:1 [barqode] calling getUserMedia
scanner.CcqxawNJ.js:2 [barqode] will use native BarcodeDetector.
barqode-stream.C4fzdaJK.js:1 [barqode] waiting for video element to load
barqode-stream.C4fzdaJK.js:1 [barqode] video element loaded
barqode-stream.C4fzdaJK.js:1 [barqode] camera ready
scanner.CcqxawNJ.js:2 [barqode] start scanning
scanner.CcqxawNJ.js:2 [barqode] will use native BarcodeDetector.

[ollema]

wait, so it doesn’t work with the native API?

that is very unfortunate. I’m not sure how to best proceed then, will have to think about this…

[joleeee]

Does it work for you? If it's a chrome bug then they might be able to fix it. I don't know how the API works, for all i know returning "" might be the equivalent of "not implemented" which is not really a "bug" bug on their side.

Alternatively, could allow us to toggle using the polyfill even if it's not required

[ollema]

what I mean was if it's not working with the native API then it's much harder to fix since we cannot really change how the built in native browser APIs work.

we might have to expose a setting for always using the polyfill as you suggested, will look into that. or happy to review a PR if you interested to contribute