From 75682268b7ed34d6216259f69f77e32c023a186c Mon Sep 17 00:00:00 2001
From: Thomas Constans <thomas@opendoor.fr>
Date: Wed, 24 Sep 2025 08:59:37 +0200
Subject: [PATCH] add var to set owner of pla install dir

---
 README.md         | 1 +
 defaults/main.yml | 1 +
 tasks/main.yml    | 7 ++++++-
 3 files changed, 8 insertions(+), 1 deletion(-)

diff --git a/README.md b/README.md
index 4bdea93..46faca9 100644
--- a/README.md
+++ b/README.md
@@ -17,6 +17,7 @@ pla_basedn - no default
 pla_install_dir - default /var/www/html
 pla_version - default 1.2.6.7
 pla_ldap_server - default {{ localhost }}
+pla_install_dir_owner - default pla, should probably be "{{ php_fpm_pool_user }}"
 
 Dependencies
 ------------
diff --git a/defaults/main.yml b/defaults/main.yml
index 438ce01..cfb2b76 100644
--- a/defaults/main.yml
+++ b/defaults/main.yml
@@ -2,4 +2,5 @@
 # defaults file for phpldapadmin
 pla_version: 1.2.6.7
 pla_install_dir: /var/www/html/
+pla_install_dir_owner: pla
 pla_ldap_server: localhost
\ No newline at end of file
diff --git a/tasks/main.yml b/tasks/main.yml
index 1c76896..318b3e0 100644
--- a/tasks/main.yml
+++ b/tasks/main.yml
@@ -11,10 +11,14 @@
     dest: /{{ pla_install_dir }}
     src:  /tmp/pla_{{pla_version }}.gz
     remote_src: true
-- ansible.builtin.file:
+
+- name: "[pla : link ]"
+  tags: skip
+  ansible.builtin.file:
     src: /{{ pla_install_dir }}/phpLDAPadmin-{{ pla_version }}
     state: link
     dest: /{{ pla_install_dir }}/pla
+    owner: "{{ pla_install_dir_owner }}"
 
 - name: "[pla] - configure"
   vars:
@@ -22,4 +26,5 @@
   ansible.builtin.template:
     src: pla_config.php
     dest: /{{ pla_install_dir }}/pla/config/config.php
+    owner: "{{ pla_install_dir_owner }}"