DIYIDP: Make the role compatible with the new mariadb docker

OpenConext · Nov 14, 2024 · 9b1ccfe · 9b1ccfe
1 parent 986646e
commit 9b1ccfe
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 5 deletions.
diff --git a/roles/diyidp/defaults/main.yml b/roles/diyidp/defaults/main.yml
@@ -14,3 +14,5 @@ diyidp_remotesp:
   - name: "{{ instance_name }} SP metadata"
     metadataurl: "https://engine.{{ base_domain }}/authentication/sp/metadata"
     acslocation: "https://engine.{{ base_domain }}/authentication/sp/consume-assertion"
+diyidp_docker_networks:
+  - name: "loadbalancer"
diff --git a/roles/diyidp/tasks/main.yml b/roles/diyidp/tasks/main.yml
@@ -74,8 +74,7 @@
     pull: true
     restart_policy: "always"
     state: started
-    networks:
-      - name: "loadbalancer"
+    networks: "{{ diyidp_docker_networks }}"
     labels:
       traefik.http.routers.diyidp.rule: "Host(`{{ diyidp_domain }}`)"
       traefik.http.routers.diyidp.tls: "true"

diff --git a/roles/diyidp/templates/config-override.php.j2 b/roles/diyidp/templates/config-override.php.j2
@@ -2,5 +2,5 @@
 $config['certdir'] = 'config/cert/';
 $config['metadatadir'] = 'config/metadata/';
 $config['auth.adminpassword'] = '{{ diyidp.admin_password }}';
-$config['baseurlpath'] = 'https://diyidp.{{ base_domain }}/';
-$config['theme.use']   = '{{ diyidp.theme |default("default") }}';
+$config['baseurlpath'] = 'https://{{ diyidp_domain }}/';
+$config['theme.use']   = '{{ diyidp_theme |default("default") }}';
diff --git a/roles/diyidp/templates/saml20-idp-hosted.php.j2 b/roles/diyidp/templates/saml20-idp-hosted.php.j2
@@ -5,7 +5,7 @@
  * See: https://rnd.feide.no/content/idp-hosted-metadata-reference
  */
 
-$metadata['https://diyidp.{{ base_domain}}'] = array(
+$metadata['https://{{ diyidp_domain}}'] = array(
 	/*
 	 * The hostname of the server (VHOST) that will use this SAML entity.
 	 *