From 66816a3189038b1edae43ddc5c834dba78ce204d Mon Sep 17 00:00:00 2001
From: Dominik Schmidt <dev@dominik-schmidt.de>
Date: Thu, 31 Jul 2025 17:58:46 +0200
Subject: [PATCH] feat: make ldap-server volumes configurable

---
 .env.example         | 3 +++
 idm/external-idp.yml | 5 +++--
 2 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/.env.example b/.env.example
index e4ed18e..7d4bcdd 100644
--- a/.env.example
+++ b/.env.example
@@ -114,6 +114,9 @@ LOG_LEVEL=
 # NOTE: you need to restart the openCloud container to load the new extensions.
 # OC_APPS_DIR=/your/local/opencloud/apps
 
+# If you're using an external IdP and don't want to use standard docker volumes for ldap-server, you can define local paths here
+# LDAP_CERTS_DIR=
+# LDAP_DATA_DIR=
 
 # S3 Storage configuration - optional
 # OpenCloud supports S3 storage as primary storage.
diff --git a/idm/external-idp.yml b/idm/external-idp.yml
index 0f18ea9..d625752 100644
--- a/idm/external-idp.yml
+++ b/idm/external-idp.yml
@@ -66,12 +66,13 @@ services:
       # Use the custom schema from opencloud because we are in full control of the ldap server
       - ./config/ldap/schemas/10_opencloud_schema.ldif:/schemas/10_opencloud_schema.ldif
       - ./config/ldap/docker-entrypoint-override.sh:/opt/bitnami/scripts/openldap/docker-entrypoint-override.sh
-      - ldap-certs:/opt/bitnami/openldap/share
-      - ldap-data:/bitnami/openldap
+      - ${LDAP_CERTS_DIR:-ldap-certs}:/opt/bitnami/openldap/share
+      - ${LDAP_DATA_DIR:-ldap-data}:/bitnami/openldap
   keycloak:
     volumes:
       - "./config/keycloak/docker-entrypoint-override.sh:/opt/keycloak/bin/docker-entrypoint-override.sh"
       - "./config/keycloak/opencloud-realm-autoprovisioning.dist.json:/opt/keycloak/data/import-dist/opencloud-realm.json"
+
 volumes:
   ldap-certs:
   ldap-data: