[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#701795: marked as done (unblock: otrs2/3.1.7+dfsg1-7)

Your message dated Tue, 12 Mar 2013 20:01:53 +0000
with message-id <20130312200153.GM5712@ernie.home.powdarrmonkey.net>
and subject line Re: Bug#701795: unblock: otrs2/3.1.7+dfsg1-7
has caused the Debian Bug report #701795,
regarding unblock: otrs2/3.1.7+dfsg1-7
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@bugs.debian.org
immediately.)


-- 
701795: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=701795
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message --- 
Package: release.debian.org
Severity: normal
User: release.debian.org@packages.debian.org
Usertags: unblock

Please unblock package otrs2

It fixes RC bug #700897:


diff -Naur '--exclude=.svn' 3.1.7+dfsg1-6/debian/changelog 3.1.7+dfsg1-7/debian/changelog
--- 3.1.7+dfsg1-6/debian/changelog      2012-10-16 11:14:16.498983306 +0200
+++ 3.1.7+dfsg1-7/debian/changelog      2013-02-27 10:25:48.144232210 +0100
@@ -1,3 +1,12 @@
+otrs2 (3.1.7+dfsg1-7) unstable; urgency=high
+
+  * Do not call otrs.SetPermissions.pl in postinst, since it modificates a few
+    files in /usr/share/otrs, instead set the required permissions in
+    /var/lib/otrs on my own.
+    Closes: #700897
+
+ -- Patrick Matthäi <pmatthaei@debian.org>  Wed, 27 Feb 2013 10:13:53 +0100
+
 otrs2 (3.1.7+dfsg1-6) unstable; urgency=medium

   * Add upstream patch 30-osa-2012-03-js-xss to improve HTML security, where a
diff -Naur '--exclude=.svn' 3.1.7+dfsg1-6/debian/otrs2.postinst 3.1.7+dfsg1-7/debian/otrs2.postinst
--- 3.1.7+dfsg1-6/debian/otrs2.postinst 2012-10-16 11:14:16.498983306 +0200
+++ 3.1.7+dfsg1-7/debian/otrs2.postinst 2013-02-27 10:25:48.144232210 +0100
@@ -86,8 +86,6 @@
                        cd $OTRSHOME && ./bin/otrs.RebuildConfig.pl > /dev/null
                        cd $OTRSHOME && ./bin/otrs.DeleteCache.pl > /dev/null
                fi
-               $OTRSHOME/bin/otrs.SetPermissions.pl --otrs-user=otrs --otrs-group=www-data \
-                       --web-user=www-data --web-group=www-data --secure $OTRSHOME > /dev/null 2>&1
                if [ -f $OTRS_VAR/Config/Files/ZZZAAuto.pm ]; then
                        chown root:www-data $OTRS_VAR/Config/Files/ZZZAAuto.pm
                fi
@@ -97,6 +95,24 @@
                chmod 0775 $OTRS_VAR/Config/
                chmod 0775 $OTRS_VAR/Config/Files
                chmod 0775 /var/spool/otrs
+               # Set required permissions, which were set by otrs.SetPermissions.pl in the past
+               if [ ! -f $OTRS_VAR/log/TicketCounter.log ]; then
+                       echo '' > $OTRS_VAR/log/TicketCounter.log
+                        chown otrs:www-data $OTRS_VAR/log/TicketCounter.log
+                        chmod 0664 $OTRS_VAR/log/TicketCounter.log
+                fi
+               chown otrs:www-data $OTRS_VAR/article
+               chmod 2775 $OTRS_VAR/article
+               chown otrs:www-data $OTRS_VAR/log
+               chmod 2775 $OTRS_VAR/log
+               chown otrs:www-data $OTRS_VAR/sessions
+                chmod 2775 $OTRS_VAR/sessions
+               chown otrs:www-data -R $OTRS_VAR/stats
+               chmod 2775 $OTRS_VAR/stats
+               chown otrs:www-data -R $OTRS_VAR/tmp
+               find $OTRS_VAR/tmp -type d -exec chmod 2775 {} \;
+               find $OTRS_VAR/httpd/htdocs/ -type d -name \*-cache -exec chown otrs:www-data {} \;
+               find $OTRS_VAR/httpd/htdocs/ -type d -name \*-cache -exec chmod 2775 {} \;
        ;;
        install)
        ;;


unblock otrs2/3.1.7+dfsg1-7

-- System Information:
Debian Release: 7.0
  APT prefers testing
  APT policy: (500, 'testing')
Architecture: amd64 (x86_64)

Kernel: Linux 3.2.0-4-amd64 (SMP w/2 CPU cores)
Locale: LANG=de_DE.UTF-8, LC_CTYPE=de_DE.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

--- End Message ---
--- Begin Message --- 
On Sat, Mar 02, 2013 at 06:32:36PM +0100, Patrick Matthäi wrote:
> Am 02.03.2013 19:17, schrieb Jonathan Wiltshire:
> >Control: tag -1 + moreinfo
> >
> >On Wed, Feb 27, 2013 at 10:32:53AM +0100, Patrick Matthäi wrote:
> >>Package: release.debian.org
> >>Severity: normal
> >>User: release.debian.org@packages.debian.org
> >>Usertags: unblock
> >>
> >>Please unblock package otrs2
> >>
> >>It fixes RC bug #700897:
> >
> >The patch looks reasonable at a glance, but why not patch
> >otrs.SetPermissions.pl in case it gets run by the user at a later date?
> >
> 
> Because SetPermission.pl is still required (for the user) if he
> wants to use the OTRS package manager, where also stuff in
> /usr/share/otrs (may; => depends on the extension) require rw
> access, as described in README.Debian.
> Also I have got more control in further versions what ACLs are set
> or not set.

Ok, unblocked. 

Thanks,

-- 
Jonathan Wiltshire                                      jmw@debian.org
Debian Developer                         http://people.debian.org/~jmw

4096R: 0xD3524C51 / 0A55 B7C5 1223 3942 86EC  74C3 5394 479D D352 4C51

Attachment: signature.asc
Description: Digital signature


--- End Message ---
Reply to: