mirror of
https://github.com/pocket-id/pocket-id.git
synced 2025-12-06 09:13:19 +03:00
🐛 Bug Report: Unable to upload oidc-clients images in distroless image #6
Reference in New Issue
Block a user
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @Skaronator on GitHub.
Reproduction steps
https://cdn.jsdelivr.net/gh/homarr-labs/dashboard-icons/svg/immich.svg)The log says:
Expected behavior
I should be able to create application with images.
Actual Behavior
Looking at the files on my system, I can see that they use UID/GID 65532.
(Sidenote: PUID/PGID env variable doesn't work with the distroless image, but this has nothing to do with this issue)
Digging deeper, I can see that the
oidc-client-imagesfolder is missing.After creating that folder manually, everything works as expected.
Pocket ID Version
ghcr.io/pocket-id/pocket-id:v1.12.0-distroless@sha256:0c1ae051060157fdb05d71bca2a28efab5417fb6e0631898b7cdaa1745b3fe6d
Database
SQLite
OS and Environment
Kubernetes 1.33 (k0s) on Debian 13 host with Envoy Gateway as Proxy.
Log Output
No response
@TheoDurr commented on GitHub:
Seems like a check of the directory existence is missing in oidc_service.go.
Similar to the check made for users profile pictures, it is required to create the directory if it's not already present.
See user_service.go
f9839a978c/backend/internal/service/user_service.go (L173-L178)Hope my fast analysis can help !
@Skaronator commented on GitHub:
Thanks for the quick fix! @stonith404
Really like Pocket ID so far. I tested a few other OIDC providers, and they require so much other stuff (redis, db, several microservices, lots of configuration) which seems way overkill for my homelab.
@janreinhardt commented on GitHub:
Unfortunately,its still not working for me on 1.13.0
This is the log:
Oct 6 11:51:31 ERR Request with errors: Error #01: failed to download logo: failed to rename file 'data/uploads/oidc-client-images/77b8e886-c8f3-4147-acb4-7a09223b675e.svg': rename data/uploads/oidc-client-images/77b8e886-c8f3-4147-acb4-7a09223b675e.svg.58518fdc-2cf5-4a4f-a88f-6288440462e4-tmp data/uploads/oidc-client-images/77b8e886-c8f3-4147-acb4-7a09223b675e.svg: file exists@stonith404 commented on GitHub:
Thanks for your detailed report, I really appreciate that. This should be fixed with
ad8a90c839and will be available in the next release.