Fix Matomo bootstrap logic and dependency ordering to prevent recursion (see conversation: https://chatgpt.com/share/692cb23b-fb3c-800f-98b6-ce7f61595305 )

This commit introduces several improvements to the Matomo initialization workflow:

- Moves the Matomo reachability check into sys-front-inj-all/tasks/01_dependencies.yml.
- Ensures web-app-matomo is only initialized when the endpoint is unreachable.
- Avoids condition inheritance on include_role by removing block-level when conditions.
- Adds explicit guarding conditions (inj_enabled.matomo, run_once_web_app_matomo usage).
- Ensures Matomo-dependent injections (CSS, Desktop, Logout, CDN) are skipped for Matomo itself.
- Fixes incorrect status_code format (now using YAML list format).
- Moves utils/run_once.yml to the top of 01_core.yml to prevent recursive re-invocation of web-app-matomo.
- Cleans Matomo config/main.yml feature definitions and clarifies which features are disabled during initial bootstrap.
- Removes legacy global Matomo bootstrap from 02_server.yml, centralizing logic in sys-front-inj-all.
- Fixes typo in inj_enabled task name.

This results in a robust, idempotent, recursion-safe Matomo bootstrap sequence that works across all injected web domains.
This commit is contained in:
2025-11-30 22:08:46 +01:00
parent 8d3874f432
commit edec4f3722
5 changed files with 49 additions and 30 deletions

View File

@@ -12,12 +12,6 @@
- sys-ctl-hlth-btrfs
- sys-ctl-rpr-btrfs-balancer
# It is necessary to setup Matomo first, because all other web apps need it if matomo is activated
- name: setup web-app-matomo
when: ('web-app-matomo' | application_allowed(group_names, allowed_applications))
include_role:
name: web-app-matomo
- name: "Include server roles"
include_tasks: "./tasks/groups/{{ item }}-roles.yml"
loop: