Bug#429866: courier-authdaemon: please add support for /var/run on tmpfs

[Stuart Pook]

hello

I had added my own support for running courier-authdaemon with /var/run on 
tmpfs. I saw that someone had added this support to the standard version so I 
removed my support. This is what the standard version contains:

if [ ! -d $rundir ]; then
		mkdir -p -m 0750 $rundir 
	fi


This is incorrect as $(dirname $rundir) must belong to daemon otherwise 
courierfilter doesn't start and all email is refused.

Apr  6 08:17:18 info courierfilter: /var/run/courier/courierfilter.pid: 
Permission denied
Apr  6 08:17:18 info courierd: Purging /var/lib/courier/msgs
Apr  6 08:17:18 info courierd: Waiting.  shutdown time=Mon Apr  6 09:17:18 
2009, wakeup time=Mon Apr  6 09:17:18 2009, queuedelivering=0, inprogress=0
Apr  6 08:17:37 info courieresmtpd: started,ip=[:::xxx.xxx.xxx.xxx]
Apr  6 08:17:38 err courieresmtpd: authdaemon: s_connect() failed: Permission 
denied

I modified the above to

if [ ! -d $rundir ]; then
if [ ! -d $(dirname $rundir) ]; then
mkdir -p -m 0750 $(dirname $rundir)
chown daemon.daemon $(dirname $rundir)
fi
		mkdir -m 0750 $rundir 
		chown daemon.daemon $rundir

fi

Perhaps the chown could be limited to /var/run/courier as I cannot see any 
non-root files in /var/run/courier/authdaemon

Stuart
--
If the From address bounces, please see http://www.pook.it/.



--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org

Bug#429866: courier-authdaemon: please add support for /var/run on tmpfs

[iroquoi]

Hi, I've just been hit by this bug.

 Sorry, I don't believe in this kind of workaround.

You don't believe in checking to see if a directory exists before trying 
to write a file into it?

 As long as there is no consensus on this matter, I won't act upon this 
 bug.

Have you tried to reach a consensus? The initscripts maintainer seems to 
believe it's the responsibility of each package to correctly handle an 
empty /var/run - see bug #422257, which refers to courier-authdaemon.

 Altogether, it would be much better to handle this in a central place 
 for all packages instead of hacking every init script.

It's been almost two years, and this central place doesn't appear to 
exist. Until the day a perfect solution is found, could you please 
reconsider adding a simple 'mkdir -p' to fix this problem?

Many thanks.



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org

Bug#429866: courier-authdaemon: please add support for /var/run on tmpfs

[Stefan Hornburg]

Andreas Beckmann wrote:

Package: courier-authdaemon
Version: 0.59.3-1
Severity: wishlist
Tags: patch

Hi,

please add support for having /var/run on tmpfs. All that's needed is to
create /var/run/courier and /var/run/courier/authdaemon in the init
script if they are missing.
A possible solution is the patch from ubuntu (attached).


Sorry, I don't believe in this kind of workaround. As long as there is 
no consensus on this matter, I won't act upon this bug.


Altogether, it would be much better to handle this in a central place 
for all

packages instead of hacking every init script.

Regards
Racke


--
LinuXia Systems = http://www.linuxia.de/
Expert Interchange Consulting and System Administration
ICDEVGROUP = http://www.icdevgroup.org/
Interchange Development Team



--
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]

Bug#429866: courier-authdaemon: please add support for /var/run on tmpfs

[Andreas Beckmann]

Package: courier-authdaemon
Version: 0.59.3-1
Severity: wishlist
Tags: patch

Hi,

please add support for having /var/run on tmpfs. All that's needed is to
create /var/run/courier and /var/run/courier/authdaemon in the init
script if they are missing.
A possible solution is the patch from ubuntu (attached).

Thanks.

Andreas

-- System Information:
Debian Release: lenny/sid
  APT prefers testing
  APT policy: (500, 'testing'), (500, 'stable'), (300, 'unstable'), (130, 
'experimental')
Architecture: i386 (i686)

Kernel: Linux 2.6.21-1-686 (SMP w/1 CPU core)
Locale: LANG=C, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/bash
diff -pruN 0.59.3-1/debian/changelog 0.59.3-1ubuntu1/debian/changelog
--- 0.59.3-1/debian/changelog   2007-05-07 17:48:11.0 +0100
+++ 0.59.3-1ubuntu1/debian/changelog2007-05-07 16:44:52.0 +0100
@@ -1,3 +1,12 @@
+courier-authlib (0.59.3-1ubuntu1) gutsy; urgency=low
+
+  * Merge from Debian unstable. Remaining Ubuntu changes:
+- debian/courier-authdaemon.init: recreate
+  /var/run/courier/authdaemon
+  * DebianMaintainerField update
+
+ -- Soren Hansen [EMAIL PROTECTED]  Tue, 1 May 2007 13:42:10 +0200
+
 courier-authlib (0.59.3-1) unstable; urgency=low
 
   * new upstream release
@@ -10,6 +19,14 @@ courier-authlib (0.59.1-0.1) experimenta
 
  -- Stefan Hornburg (Racke) [EMAIL PROTECTED]  Wed, 17 Jan 2007 11:58:29 
+0100
 
+courier-authlib (0.58-5ubuntu1) feisty; urgency=low
+
+  * Merge from Debian unstable, remaining change:
++ debian/courier-authdaemon.init: recreate
+  /var/run/courier/authdaemon
+
+ -- Soren Hansen [EMAIL PROTECTED]  Sat,  2 Dec 2006 03:35:16 +0100
+
 courier-authlib (0.58-5) unstable; urgency=low
 
   * added dependency and build dependency on expect (Closes: #400812,
@@ -17,6 +34,14 @@ courier-authlib (0.58-5) unstable; urgen
 
  -- Stefan Hornburg (Racke) [EMAIL PROTECTED]  Wed, 29 Nov 2006 11:42:45 
+0100
 
+courier-authlib (0.58-4ubuntu1) edgy; urgency=low
+
+  * Merge from Debian unstable. Remaining changes:
++ debian/courier-authdaemon.init: recreate
+  /var/run/courier/authdaemon
+
+ -- Michael Bienia [EMAIL PROTECTED]  Wed, 27 Sep 2006 00:58:55 +0200
+
 courier-authlib (0.58-4) unstable; urgency=medium
 
   * call dh_makeshlibs during binary-arch target in order to get proper shlib
@@ -37,6 +62,13 @@ courier-authlib (0.58-3.1) unstable; urg
 
  -- Margarita Manterola [EMAIL PROTECTED]  Tue,  1 Aug 2006 16:45:07 -0300
 
+courier-authlib (0.58-3ubuntu1) edgy; urgency=low
+
+  * Initial upload to Ubuntu:
+- replace init script with LSB init script from original courier source.
+
+ -- Scott James Remnant [EMAIL PROTECTED]  Tue, 11 Jul 2006 21:00:57 +0100
+
 courier-authlib (0.58-3) unstable; urgency=low
 
   * remove all Courier runtime files on purge of courier-authdaemon
diff -pruN 0.59.3-1/debian/control 0.59.3-1ubuntu1/debian/control
--- 0.59.3-1/debian/control 2007-05-07 17:48:11.0 +0100
+++ 0.59.3-1ubuntu1/debian/control  2007-05-07 16:44:52.0 +0100
@@ -1,7 +1,8 @@
 Source: courier-authlib
 Section: mail
 Priority: optional
-Maintainer: Stefan Hornburg (Racke) [EMAIL PROTECTED]
+Maintainer: Ubuntu MOTU Developers [EMAIL PROTECTED]
+XSBC-Original-Maintainer: Stefan Hornburg (Racke) [EMAIL PROTECTED]
 Standards-Version: 3.6.1.1
 Build-Depends: libmysqlclient-dev | libmysqlclient15-dev, libpam0g-dev, 
libgdbm-dev | libgdbmg1-dev, debhelper (= 4.1.16), libldap-dev, libsasl2-dev | 
libsasl-dev, libpq-dev | postgresql-dev, libtool, libltdl3-dev, procps, expect
 
diff -pruN 0.59.3-1/debian/courier-authdaemon.init 
0.59.3-1ubuntu1/debian/courier-authdaemon.init
--- 0.59.3-1/debian/courier-authdaemon.init 2007-05-07 17:48:11.0 
+0100
+++ 0.59.3-1ubuntu1/debian/courier-authdaemon.init  2007-05-07 
16:44:52.0 +0100
@@ -5,6 +5,14 @@ exec_prefix=${prefix}
 sysconfdir=/etc/courier
 sbindir=${exec_prefix}/sbin
 daemonscript=${sbindir}/authdaemond
+run_dir=/var/run/courier/authdaemon
+
+# create  /var/run if it does not already exist
+if [ ! -d ${run_dir} ]; then
+   mkdir -p ${run_dir}
+   chown daemon:daemon ${run_dir} /var/run/courier
+   chmod 750 ${run_dir}
+fi
 
 . /lib/lsb/init-functions