damongolding / immich-kiosk

Immich Kiosk is a lightweight slideshow for running on kiosk devices and browsers that uses Immich as a data source.
GNU Affero General Public License v3.0
532 stars 19 forks source link

Error processing image #127

Closed soana closed 1 month ago

soana commented 1 month ago

Describe the bug Per the error message it looks like app has an issue communicating with the immich server

Screenshot 2024-10-10 at 6 09 03 PM

To Reproduce After an update that was applied today the issue started happening right away

Your Kiosk version Version 0.11.2

Your Kiosk installation

Your Kiosk ENV or config file <?xml version="1.0"?>

immich-kiosk damongolding/immich-kiosk:latest https://hub.docker.com/r/damongolding/immich-kiosk/ bridge sh false https://hub.docker.com/r/damongolding/immich-kiosk/ https://github.com/damongolding/immich-kiosk A kiosk for your Immich images Converted By Community Applications Always verify this template (and values) against the support page for the container https://hub.docker.com/r/damongolding/immich-kiosk/ repository official: damongolding/immich-kiosk:latest repository development damongolding/immich-kiosk-development:latest MediaApp:Photos http://[IP]:[PORT:3000] https://github.com/damongolding/immich-kiosk/blob/main/assets/logo.png?raw=true 1728598021 3000 true America/Detroit *********************************** http://***.***.***.***:8088 FALSE TRUE DD MMM YYYY TRUE 24 30 ********-****-****-****-*********** ********-****-****-****-***********, ********-****-****-****-***********, ********-****-****-****-*********** CONTAIN TRUE FADE TRUE TRUE 24 TRUE DD MMM YYYY true 23 630 200 fade splitview

Any parms passed to the URL used to access Kiosk

Raspberry Pi:

MacOS

Additional context I can confirm that Immich server is OK and running and that the albums and persons are present on the Immich side

For some reason the docker xml file does not up properly, below are some screenshots:

Screenshot 2024-10-10 at 6 42 40 PM Screenshot 2024-10-10 at 6 42 59 PM Screenshot 2024-10-10 at 6 42 51 PM Screenshot 2024-10-10 at 6 42 25 PM Screenshot 2024-10-10 at 6 42 13 PM Screenshot 2024-10-10 at 6 42 01 PM
damongolding commented 1 month ago

It looks like person and album are empty in the unraid GUI but have values in the output above 🤔. Are you adding either albums or people?

I should also note that I have no idea who created and manages this unraid template.

damongolding commented 1 month ago

But in essence the error stems from Kiosk trying to retrieve a person's asset count from Immich to use for asset weighting.

In this case a status of 400 has been sent back from Immich, which means the request was malformed. Kiosk expects something between 200-300 for a successful response.

Is the person ID correct? It can change if you make certain changes to a person I.e. merge two people.

I get the same error if I set person to a ID that doesn't exist.

soana commented 1 month ago

Both album and person fields are there I just wiped them off for the screenshots, this is my personal template I created before an template was available in the unraid apps store.

After several trails I figured out that the issue comes from the KIOSK_PERSON variable. Up until recently it used to work with a space after the comma separating multiple persons but lately that did not work anymore giving the error. Removing the extra space between person id's fixed the issue.

NG: 83fdb2e4-a941-423d-8a72-bb46319afdf1, 1e067673-9e0a-446f-9b06-70fa32c73970 OK: 83fdb2e4-a941-423d-8a72-bb46319afdf1,1e067673-9e0a-446f-9b06-70fa32c73970

damongolding commented 1 month ago

Ahh ok. Nice debugging!

I’m going to patch this now. You’ll be able to use the space again (if you wish) after the patch.

damongolding commented 1 month ago

Patched in 0.11.3