diff options
| author | Micah Anderson <micah@riseup.net> | 2006-06-10 00:33:51 +0000 |
|---|---|---|
| committer | Micah Anderson <micah@riseup.net> | 2006-06-10 00:33:51 +0000 |
| commit | fe7ee521a660ef38e5657c31eefb14b95f22827f (patch) | |
| tree | 8197ea749de4fa06a75e81fef97196d0f3ade44b | |
| parent | a2436c2ba4abc165b8300578b9fd80437b9bc201 (diff) | |
| download | backupninja-fe7ee521a660ef38e5657c31eefb14b95f22827f.tar.gz backupninja-fe7ee521a660ef38e5657c31eefb14b95f22827f.tar.bz2 | |
add in-line compression to ldap handler
| -rw-r--r-- | ChangeLog | 6 | ||||
| -rw-r--r-- | handlers/ldap | 22 |
2 files changed, 19 insertions, 9 deletions
@@ -15,13 +15,13 @@ version 0.9.4 -- unreleased . Fixed erroneous removal of tmpfile when it didn't exit . Fixed inversed vsname emptiness check . Fixed su quote usage to be more posixy - . compress for sqldumps now happens in-line to save some disk space (Closes: #370778) + . Compress for sqldumps now happens in-line to save some disk space (Closes: #370778) pgsql: . Fixed inversed vsname emptiness check . Fixed su quote usage to be more posixy . Fixed shell expansion, thanks Thomas Kotzian (Closes: #363297) . postgres user UID is now the one from inside the vserver if necessary - . compress now happens in-line to save some disk space (Closes: #370778) + . Compress now happens in-line to save some disk space (Closes: #370778) svn: . Fixed inversed vsname emptiness check rdiff: @@ -43,6 +43,8 @@ version 0.9.4 -- unreleased . Now forbid to (try to) include /. sys: . Many more system checks were added, thanks to Petr Klíma + ldap: + . Compress now happens in-line to save some disk space (Closes: #370778) lib changes vserver: . init_vservers: fixed Debian bug #351083 (improper readlink syntax) diff --git a/handlers/ldap b/handlers/ldap index 7492e69..ee46831 100644 --- a/handlers/ldap +++ b/handlers/ldap @@ -46,10 +46,18 @@ if [ "$ldif" == "yes" ]; then fi if [ "$method" == "slapcat" ]; then - execstr="$SLAPCAT -f $conf -b $dbsuffix" + if [ "$compress" == "yes" ]; then + execstr="$SLAPCAT -f $conf -b $dbsuffix | $GZIP" + else + execstr="$SLAPCAT -f $conf -b $dbsuffix" + fi debug "$execstr" else - execstr="$LDAPSEARCH -x -L -b ""$dbsuffix"" -D ""$binddn"" -y $passwordfile" + if [ "$compress" == "yes" ]; then + execstr="$LDAPSEARCH -x -L -b ""$dbsuffix"" -D ""$binddn"" -y $passwordfile | $GZIP" + else + execstr="$LDAPSEARCH -x -L -b ""$dbsuffix"" -D ""$binddn"" -y $passwordfile" + fi [ -f "$passwordfile" ] || fatal "Password file $passwordfile not found. When method is set to ldapsearch, you must also specify a password file." debug "$execstr" fi @@ -64,7 +72,11 @@ if [ "$ldif" == "yes" ]; then fatal "Couldn't create ldif dump file: $dumpdir/$dbsuffix.ldif" fi - output=`$execstr > $dumpdir/$dbsuffix.ldif` + if [ "$compress" == "yes" ]; then + output=`$execstr > $dumpdir/$dbsuffix.ldif.gz` + else + output=`$execstr > $dumpdir/$dbsuffix.ldif` + fi code=$? if [ "$code" == "0" ]; then debug $output @@ -73,10 +85,6 @@ if [ "$ldif" == "yes" ]; then warning $output warning "Failed ldif export of $dbsuffix" fi - if [ "$compress" == "yes" ]; then - output=`$GZIP -f "$dumpdir/$dbsuffix.ldif" 2>&1` - debug $output - fi if [ "$restart" == "yes" ]; then debug "Starting ldap server..." |