fix(cron): make sure cron container only returns one container id #1

Merged
simon merged 1 commits from kolaente/pretix:fix/cron into main 2025-10-02 12:06:04 +00:00
Owner

This fixes a bug where another app with the same name was matched by the cron. For example, if the pretix stack is called tickets_domain_de and there is another app with a name like traefik-tickets_domain_de the regex would match both and try to execute with the latter container (failing).

This fixes a bug where another app with the same name was matched by the cron. For example, if the pretix stack is called `tickets_domain_de` and there is another app with a name like `traefik-tickets_domain_de` the regex would match both and try to execute with the latter container (failing).
kolaente added 1 commit 2025-09-29 20:26:57 +00:00
simon approved these changes 2025-10-02 12:05:57 +00:00
simon left a comment
Owner

oh, nasty issues! tysm for the find!

oh, nasty issues! tysm for the find!
simon merged commit f277fe7070 into main 2025-10-02 12:06:04 +00:00
Author
Owner

On second thought (but I would need to check this): maybe we could do this without the docker ps and pass the docker app name directly to docker exec?

On second thought (but I would need to check this): maybe we could do this without the `docker ps` and pass the docker app name directly to `docker exec`?
kolaente deleted branch fix/cron 2025-10-02 13:07:25 +00:00
Sign in to join this conversation.
No description provided.