bayang / jelu

Self hosted read and to-read list book tracker
MIT License
366 stars 14 forks source link

Question about ISBN scanner #150

Closed DeepWoods closed 3 weeks ago

DeepWoods commented 3 weeks ago

Is there any additional settings required or something I'm missing to have the scanner/QR modal working to add books? I'm running the latest(0.60.1) image with docker compose and have also pulled 0.56.2 and 0.59.2 and rebuilt the containers but all with the same problem that seems to be similar to issue #118 again. Testing from that referenced upstream repo via their demo works well though so I don't believe it's the same issue and maybe something to do with my settings. I've tested with several android(12 & 14) devices and an older iPad(15.8.3), all of them do not ask for camera permissions or show the camera view and the screen stays stuck on the "Import Book" overlay with reloading or leaving the page the only way to dismiss it. The only thing in my application.yml is the metadataproviders for google books API and debugging. I've tested with and without an application.yml file. Are there any extra steps or settings I might be missing in the compose file to get the scanner working to add books?

bayang commented 3 weeks ago

No there should not be anything to do to make it work. No config associated. I just tried in firefox, chrome on desktop and on android. To be fair this lib has never worked really well, I am searching for a replacement. There are some clues on the origin repo like trusted sites, vulkan etc... https://github.com/teckel12/vue-barcode-reader/issues/7

DeepWoods commented 3 weeks ago

Screen Shot 2024-10-27 at 12 58 40 PM The above screenshot is from chrome on my MB Air after adding the docker container IP to the insecure sites.
I also tried enabling the Vulkan engine and adding the docker container IP to the insecure sites on an android phone without any change. I've tried Brave and Vivaldi browsers as well but they're all chrome/chromium engines anyway. Below is the error with Brave but I'm not adept enough with Vue/JS to troubleshoot further myself. Screen Shot 2024-10-27 at 1 04 38 PM

bayang commented 3 weeks ago

The error message in the browser console and the address of your app make me think you probably in the same situation as this issue https://github.com/olefirenko/vue-barcode-reader/issues/4

DeepWoods commented 3 weeks ago

Quite possible! A quick check in site settings shows the camera as blocked in the screenshot below. I'll see if I can bypass that somehow. Screen Shot 2024-10-27 at 2 00 02 PM

DeepWoods commented 3 weeks ago

Evidently I was wrong when I added just the IP of my container to the insecure sites chrome flag. I also needed to include the port in the URL like the screenshot below and now the camera requests permission and scanning works as expected. I also still have the Vulcan engine enabled on android along with the insecure site setting just for reference. Thanks @bayang for helping me resolve an issue that turned out to be a settings problem on my end.
Screen Shot 2024-10-27 at 2 19 14 PM

bayang commented 3 weeks ago

Evidently I was wrong when I added just the IP of my container to the insecure sites chrome flag. I also needed to include the port in the URL like the screenshot below and now the camera requests permission and scanning works as expected. I also still have the Vulcan engine enabled on android along with the insecure site setting just for reference. Thanks @bayang for helping me resolve an issue that turned out to be a settings problem on my end. Screen Shot 2024-10-27 at 2 19 14 PM

No problem, I'm glad you managed to make it work. Good catch !