From 1f99a6b84b66a60ead51a9ded375d052bec80b3b Mon Sep 17 00:00:00 2001 From: Kevin Veen-Birkenbach Date: Thu, 25 Sep 2025 11:55:13 +0200 Subject: [PATCH] Refactor: force early evaluation of BlueSky redirect_domain_mappings before include_role Ensures that redirect_domain_mappings is resolved via set_fact before passing it into the web-opt-rdr-domains role. See: https://chatgpt.com/share/68d51125-14f4-800f-be6a-a7be3faeb028 --- roles/web-app-bluesky/tasks/main.yml | 14 +++++++++----- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/roles/web-app-bluesky/tasks/main.yml b/roles/web-app-bluesky/tasks/main.yml index fbcfb575..1788b75f 100644 --- a/roles/web-app-bluesky/tasks/main.yml +++ b/roles/web-app-bluesky/tasks/main.yml @@ -39,12 +39,16 @@ include_tasks: "03_dns.yml" when: DNS_PROVIDER | lower == 'cloudflare' +- name: Resolve redirect_domain_mappings now (before include_role) + set_fact: + BLUESKY_REDIRECT_DOMAIN_MAPPINGS: >- + {{ + ( (BLUESKY_WEB_ENABLED | bool) | ternary([], [ {'source': BLUESKY_WEB_DOMAIN, 'target': BLUESKY_API_DOMAIN } ]) ) + + ( (BLUESKY_VIEW_ENABLED | bool) | ternary([], [ {'source': BLUESKY_VIEW_DOMAIN, 'target': BLUESKY_API_DOMAIN } ]) ) + }} + - name: "Redirect deactivated BlueSky Services to {{ BLUESKY_API_DOMAIN }}" include_role: name: web-opt-rdr-domains vars: - redirect_domain_mappings: >- - {{ - ( (BLUESKY_WEB_ENABLED | bool) | ternary([], [ {'source': BLUESKY_WEB_DOMAIN, 'target': BLUESKY_API_DOMAIN } ]) ) - + ( (BLUESKY_VIEW_ENABLED | bool) | ternary([], [ {'source': BLUESKY_VIEW_DOMAIN, 'target': BLUESKY_API_DOMAIN } ]) ) - }} \ No newline at end of file + redirect_domain_mappings: "{{ BLUESKY_REDIRECT_DOMAIN_MAPPINGS }}" \ No newline at end of file