docs & fix service dir

Author: BertrandGouny

README.md

@@ -82,8 +82,8 @@ By default HTTPS is enable, a certificate is created with the container hostname
 You can set your custom certificate at run time, by mounting a directory containing those files to **/container/service/phpldapadmin/assets/apache2/certs** and adjust their name with the following environment variables:
 
 	docker run --volume /path/to/certifates:/container/service/phpldapadmin/assets/apache2/certs \
-	--env PHPLDAPADMIN_HTTPS_CRT_FILENAME=my-phpldapadmin.crt \
-	--env PHPLDAPADMIN_HTTPS_KEY_FILENAME=my-phpldapadmin.key \
+	--env PHPLDAPADMIN_HTTPS_CRT_FILENAME=my-cert.crt \
+	--env PHPLDAPADMIN_HTTPS_KEY_FILENAME=my-cert.key \
 	--env PHPLDAPADMIN_HTTPS_CA_CRT_FILENAME=the-ca.crt \
 	--detach osixia/phpldapadmin:0.6.7
 

image/service/ldap-client/startup.sh

@@ -12,7 +12,8 @@ if [ ! -e "$FIRST_START_DONE" ]; then
 
   if [ "${PHPLDAPADMIN_LDAP_CLIENT_TLS,,}" == "true" ]; then
 
-    # check certificat and key or create it
+    # generate a certificate and key if files don't exists
+    # https://github.com/osixia/docker-light-baseimage/blob/stable/image/service-available/:cfssl/assets/tool/cfssl-helper
     cfssl-helper ${PHPLDAPADMIN_LDAP_CLIENT_CFSSL_PREFIX} "${CONTAINER_SERVICE_DIR}/ldap-client/assets/certs/${PHPLDAPADMIN_LDAP_CLIENT_TLS_CRT_FILENAME}" "${CONTAINER_SERVICE_DIR}/ldap-client/assets/certs/${PHPLDAPADMIN_LDAP_CLIENT_TLS_KEY_FILENAME}" "${CONTAINER_SERVICE_DIR}/ldap-client/assets/certs/${PHPLDAPADMIN_LDAP_CLIENT_TLS_CA_CRT_FILENAME}"
 
     # ldap client config