Merge branch 'hotfix-0.7.2' into stable

Dockerfile

@@ -9,11 +9,16 @@ ENV LDAP_ADMIN_PWD toor
 ENV LDAP_ORGANISATION Example Inc.
 ENV LDAP_DOMAIN example.com
 
-# /!\ To store the data outside the container, mount /var/lib/ldap as a data volume
+# /!\ To store the data outside the container, 
-# add -v /some/host/directory:/var/lib/ldap to the run command
+# mount /var/lib/ldap and /etc/ldap/slapd.d as a data volume add
+# -v /some/host/directory:/var/lib/ldap and -v /some/other/host/directory:/etc/ldap/slapd.d
+# to the run command
 
 # Disable SSH
-RUN rm -rf /etc/service/sshd /etc/my_init.d/00_regen_ssh_host_keys.sh
+# RUN rm -rf /etc/service/sshd /etc/my_init.d/00_regen_ssh_host_keys.sh
+
+# Enable dnsmasq
+RUN /etc/enable-service dnsmasq
 
 # Use baseimage-docker's init system.
 CMD ["/sbin/my_init"]
@@ -32,11 +37,11 @@ RUN mkdir /etc/ldap/ssl
 
 # Add config directory 
 RUN mkdir /etc/ldap/config
-ADD config /etc/ldap/config
+ADD service/slapd/config /etc/ldap/config
 
 # Add slapd deamon
 RUN mkdir /etc/service/slapd
-ADD slapd.sh /etc/service/slapd/run
+ADD service/slapd/slapd.sh /etc/service/slapd/run
 
 # Clear out the local repository of retrieved package files
 RUN apt-get clean && rm -rf /var/lib/apt/lists/* /tmp/* /var/tmp/*

LICENSE.txt

@@ -0,0 +1,21 @@
+The MIT License (MIT)
+
+Copyright (c) 2014 Osixia
+
+Permission is hereby granted, free of charge, to any person obtaining a copy
+of this software and associated documentation files (the "Software"), to deal
+in the Software without restriction, including without limitation the rights
+to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+copies of the Software, and to permit persons to whom the Software is
+furnished to do so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in all
+copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+SOFTWARE.

README.md

@@ -1,4 +1,4 @@
-## docker-openldap
+# docker-openldap
 
 Fork of Nick Stenning docker-slapd :
 https://github.com/nickstenning/docker-slapd
@@ -30,3 +30,4 @@ and optionaly `dhparam.pem` this file is genereted automaticaly if not present.
                -e LDAP_ORGANISATION="Example Corp." \
                -e LDAP_ROOTPASS=toor \
                -p 389:389 -d osixia/openldap
+

service/slapd/slapd.sh

@@ -46,11 +46,7 @@ fi
 if [ ! -e /etc/ldap/config/docker_bootstrapped ]; then
   status "Custom config"
 
-  # permission error on /etc/ldap/slapd.conf if not set?! :'(
+  slapd -h "ldapi:///" 
-  adduser openldap root
-
-  slapd -h "ldapi:///" -u openldap -g openldap 
-  chown -R openldap:openldap /etc/ldap
 
   # TLS
   if [ -e /etc/ldap/ssl/ldap.crt ] && [ -e /etc/ldap/ssl/ldap.key ] && [ -e /etc/ldap/ssl/ca.crt ]; then
@@ -88,4 +84,6 @@ fi
 
 status "starting slapd on default port 389"
 set -x
-exec /usr/sbin/slapd -h "ldap:///" -u openldap -g openldap -d -1
+#with debug:
+#exec /usr/sbin/slapd -h "ldap:///" -d -1
+exec /usr/sbin/slapd -h "ldap:///" 

test.sh

@@ -2,55 +2,14 @@
 
 # Usage
 # sudo ./test.sh 
-# add -v for verbose mode
+# add -v for verbose mode (or type whatever you like !) :p
 
-verbose=$1
+. test/tools/run.sh
-error=0
-ok=0
 
-echo_start () {
+run_test tools/build-container.sh "Successfully built"
-  echo "------- Start $* test -------"
+run_test simple.sh "dn: dc=example,dc=com"
-}
+run_test tls.sh "dn: dc=example,dc=com"
+run_test db.sh "dn: dc=otherdomain,dc=com"
 
-echo_error () {
+. test/tools/end.sh
-  echo "\n$(tput setaf 1)/!\ $* failed$(tput sgr0)\n"
-  error=`expr $error + 1`
-}
-
-echo_ok () {
-  echo "\n--> $* ok\n"
-  ok=`expr $ok + 1`
-}
-
-run_test () {
-  fction=$1
-  out=test/test.out
-
-  echo_start $fction
-
-  if [ -z ${verbose} ]; then
-    ./test/$1.sh > $out 2>&1
-  else
-    ./test/$1.sh | tee $out 2>&1
-  fi
-  
-  if [ "$(grep -c "$2" $out)" -eq 0 ]; then
-    echo_error $fction
-  else
-    echo_ok  $fction
-  fi
-
-  rm $out
-}
-
-
-./test/tools/prepare.sh > /dev/null 2>&1
-run_test build "Successfully built"
-run_test simple "dn: dc=example,dc=com"
-run_test tls "dn: dc=example,dc=com"
-run_test db "dn: dc=otherdomain,dc=com"
-./test/tools/end.sh > /dev/null 2>&1
-
-echo "------- Test finished -------"
-echo $error " failed " $ok " passed"
 

test/build.sh

@@ -1,8 +0,0 @@
-#!/bin/sh
-
-dir=$(dirname $0)
-. $dir/tools/config.prop
-
-docker.io build -t $openldapTestImage .
-#docker.io build --no-cache=true -t $openldapTestImage .
-

test/db.sh

@@ -3,21 +3,22 @@
 dir=$(dirname $0)
 . $dir/tools/config.prop
 
-if [ -d "$openldapTestDir" ]; then
+if [ -d "$testDir" ]; then
-  rm -r $openldapTestDir
+  rm -r $testDir
 fi
 
-mkdir $openldapTestDir
+mkdir $testDir
-mkdir $openldapTestDir/db
+mkdir $testDir/db
-mkdir $openldapTestDir/config
+mkdir $testDir/config
 
-runOptions="-e LDAP_DOMAIN=otherdomain.com -v $openldapTestDir/db:/var/lib/ldap -v $openldapTestDir/config:/etc/ldap/slapd.d"
+runOptions="-e LDAP_DOMAIN=otherdomain.com -v $testDir/db:/var/lib/ldap -v $testDir/config:/etc/ldap/slapd.d"
-. $dir/tools/run-simple.sh
+. $dir/tools/run-container.sh
 $dir/tools/delete-container.sh
 
-runOptions="-v $openldapTestDir/db:/var/lib/ldap -v $openldapTestDir/config:/etc/ldap/slapd.d"
+runOptions="-v $testDir/db:/var/lib/ldap -v $testDir/config:/etc/ldap/slapd.d"
-. $dir/tools/run-simple.sh
+. $dir/tools/run-container.sh
-ldapsearch -x -h localhost -p 65389 -b dc=otherdomain,dc=com
+echo "ldapsearch -x -h $IP -b dc=otherdomain,dc=com"
+ldapsearch -x -h $IP -b dc=otherdomain,dc=com
 
-rm -r $openldapTestDir
+rm -r $testDir
 $dir/tools/delete-container.sh

test/simple.sh

@@ -3,7 +3,8 @@
 dir=$(dirname $0)
 . $dir/tools/config.prop
 
-. $dir/tools/run-simple.sh
+. $dir/tools/run-container.sh
-ldapsearch -x -h localhost -p 65389 -b dc=example,dc=com
+echo "ldapsearch -x -h $IP -b dc=example,dc=com"
+ldapsearch -x -h $IP -b dc=example,dc=com
 
 $dir/tools/delete-container.sh

test/tls.sh

@@ -3,7 +3,8 @@
 dir=$(dirname $0)
 . $dir/tools/config.prop
 
-. $dir/tools/run-tls.sh
+. $dir/tls/run.sh
-ldapsearch -x -h $certCN -p 65389 -b dc=example,dc=com -ZZ
+echo "ldapsearch -x -h $certCN -b dc=example,dc=com -ZZ"
+ldapsearch -x -h $certCN -b dc=example,dc=com -ZZ
 
-. $dir/tools/end-tls.sh
+. $dir/tls/end.sh

test/tls/run.sh

@@ -1,12 +1,12 @@
 #!/bin/sh
 
 
-runOptions="--dns=127.0.0.1 -v `pwd`/test/ssl:/etc/ldap/ssl"
+runOptions="--dns=127.0.0.1 -v `pwd`/test/tls/ssl:/etc/ldap/ssl"
-. $dir/tools/run-simple.sh
+. $dir/tools/run-container.sh
 
-cert=$(echo $dir/ssl/ldap.crt)
+cert=$(echo $dir/tls/ssl/ldap.crt)
 certCN=$(openssl x509 -in $cert -subject -noout | sed -n 's/.*CN=\(.*\)\/*\(.*\)/\1/p')
-addLine=$(echo "127.0.0.1" $certCN)
+addLine=$(echo $IP $certCN)
 
 echo $addLine >> /etc/hosts
 cp /etc/ldap/ldap.conf /etc/ldap/ldap.conf.old

test/tools/build-container.sh

@@ -0,0 +1,7 @@
+#!/bin/sh
+
+. $(dirname $0)/config.prop
+
+docker.io build -t $testImage .
+#docker.io build --no-cache=true -t $testImage .
+

test/tools/config.prop

@@ -1,3 +1,3 @@
-openldapTestImage=openldap-test
+testImage=openldap-test
-openldapTestContainer=openldap-test-container
+testContainer=openldap-test-container
-openldapTestDir=/osixia-test-docker-openldap
+testDir=/osixia-test-docker-openldap

test/tools/delete-container.sh

@@ -2,10 +2,10 @@
 
 . $(dirname $0)/config.prop
 
-# remove openldap test container
+# remove test container
-res=$(docker.io ps -a | grep -c "$openldapTestContainer")
+res=$(docker.io ps -a | grep -c "$testContainer")
 
 if [ $res -ne 0 ]; then
-  docker.io stop $openldapTestContainer
+  docker.io stop $testContainer
-  docker.io rm $openldapTestContainer
+  docker.io rm $testContainer
 fi

test/tools/delete-image.sh

@@ -2,9 +2,9 @@
 
 . $(dirname $0)/config.prop
 
-# remove openldap test image
+# remove test image
-res=$(docker.io images | grep -c "$openldapTestImage")
+res=$(docker.io images | grep -c "$testImage")
 
 if [ $res -ne 0 ]; then
-  docker.io rmi $openldapTestImage
+  #docker.io rmi $testImage
 fi

test/tools/end.sh

@@ -1,5 +1,8 @@
 #!/bin/sh
 
 dir=$(dirname $0)
-$dir/delete-container.sh
+$dir/delete-container.sh > /dev/null 2>&1
-$dir/delete-image.sh
+$dir/delete-image.sh > /dev/null 2>&1
+
+echo "------- End -------"
+echo $error " failed " $ok " succeeded"

test/tools/run-container.sh

@@ -0,0 +1,14 @@
+#!/bin/sh
+
+echo "docker.io run --name $testContainer $runOptions -d $testImage"
+ID=`docker.io run --name $testContainer $runOptions -d $testImage`
+sleep 7
+
+echo " --> Obtaining IP"
+IP=`docker inspect -f "{{ .NetworkSettings.IPAddress }}" $ID`
+if [ "$IP" = "" ]; then
+	abort "Unable to obtain container IP"
+	exit 1
+else
+  echo " -->" $IP
+fi

test/tools/run-simple.sh

@@ -1,5 +0,0 @@
-#!/bin/sh
-
-echo "docker.io run --name $openldapTestContainer $runOptions -p 65389:389 -d $openldapTestImage"
-docker.io run --name $openldapTestContainer $runOptions -p 65389:389 -d $openldapTestImage
-sleep 5

test/tools/run.sh

@@ -0,0 +1,46 @@
+#!/bin/sh
+
+# Usage
+# sudo ./test.sh 
+# add -v for verbose mode (or type whatever you like !) :p
+
+verbose=$1
+error=0
+ok=0
+
+echo_start () {
+  echo "------- Test: $* -------"
+}
+
+echo_error () {
+  echo "\n$(tput setaf 1)/!\ $* failed$(tput sgr0)\n"
+  error=`expr $error + 1`
+}
+
+echo_ok () {
+  echo "\n--> $* ok\n"
+  ok=`expr $ok + 1`
+}
+
+run_test () {
+  test=$1
+  out=test/test.out
+
+  echo_start $test
+
+  if [ -z ${verbose} ]; then
+    ./test/$test > $out 2>&1
+  else
+    ./test/$test | tee $out 2>&1
+  fi
+  
+  if [ "$(grep -c "$2" $out)" -eq 0 ]; then
+    echo_error $test
+  else
+    echo_ok  $test
+  fi
+
+  rm $out
+}
+
+./test/tools/prepare.sh > /dev/null 2>&1