On Friday, August 1, 2014 4:54:18 AM UTC-7, Sylvain Raybaud wrote: > > On 29/07/2014 06:59, Ryan Ollos wrote: > > <[email protected] <javascript:> <mailto:[email protected] > <javascript:>>> wrote: > > > > Dear Trac users community, > > > > I'm facing a problem with my company trac installation. I've got a > > trac installation with a bunch of registered users (usernames and > > passwords in /var/trac/main/digest). But whoever the author of a > > changeset is he is able to close a ticket by including the > > appropriate command in the commit message. Here are the steps to > > reproduce: > > > > [...] > > > > > > You could set log level to DEBUG and post the output that you see when > > pushing the changeset and closing the ticket. > > http://trac.edgewall.org/wiki/TracLogging > > > > Which version of Trac are you running? > > > Hello > > Thanks for your reply. I'm running Trac version 1.1.2dev-r12865. > > I did set the loglevel to DEBUG and here is what shows up. I'll skip the > numerous "Trac[loader] DEBUG: Loading ..." to go directly to what seems > relevant: > > 2014-07-21 13:52:33,862 Trac[loader] DEBUG: Loading > tracopt.versioncontrol.svn.svn_prop from > /usr/local/lib/python2.7/dist-packages/Trac-1.1.2dev_r12865-py2.7.egg > > [... and more similar entries before ...] > > 2014-07-21 13:52:33,863 Trac[default_workflow] DEBUG: Workflow actions > at initialization: {u'resolve': {u'operations': [u'set_resolution'], > 'name': u'resolve', 'default': 0, 'newstate': u'clos > ed', 'oldstates': [u'new', u'assigned', u'accepted', u'reopened'], > u'permissions': [u'TICKET_MODIFY']}, u'accept': {u'operations': > [u'set_owner_to_self'], 'name': u'accept', 'default': 0, 'new > state': u'accepted', 'oldstates': [u'new', u'assigned', u'accepted', > u'reopened'], u'permissions': [u'TICKET_MODIFY']}, u'leave': > {u'operations': [u'leave_status'], 'name': u'leave', u'default > ': 1, 'newstate': u'*', 'oldstates': [u'*'], 'permissions': []}, > u'reopen': {u'operations': [u'del_resolution'], 'name': u'reopen', > 'default': 0, 'newstate': u'reopened', 'oldstates': [u'close > d'], u'permissions': [u'TICKET_CREATE']}, u'reassign': {u'operations': > [u'set_owner'], 'name': u'reassign', 'default': 0, 'newstate': > u'assigned', 'oldstates': [u'new', u'assigned', u'accepted > ', u'reopened'], u'permissions': [u'TICKET_MODIFY']}, '_reset': > {'operations': ['reset_workflow'], 'name': 'reset', 'default': 0, > 'newstate': 'new', 'oldstates': [], 'permissions': []}} > > 2014-07-21 13:52:33,867 Trac[api] DEBUG: Event changeset_added on > repository 'boost-cgi' for changesets > (u'86034d863b512607f374a0ebfd28f28f4f969662',) > 2014-07-21 13:52:33,881 Trac[git_fs] INFO: detected GIT version 1.7.10.4 > 2014-07-21 13:52:33,882 Trac[PyGIT] DEBUG: PyGIT.Storage instance > 58229328 constructed > 2014-07-21 13:52:33,882 Trac[PyGIT] DEBUG: requested weak PyGIT.Storage > instance 58229328 for '/home/git/repositories/boost-cgi.git' > 2014-07-21 13:52:33,883 Trac[git_fs] DEBUG: disabled CachedRepository > for '/home/git/repositories/boost-cgi.git' > > [... many similar lines follow which are not related to this repository > ...] > > 2014-07-21 13:52:33,932 Trac[PyGIT] DEBUG: rebuilt commit tree db for > 58229328 with 227 entries (took 31.8 ms) > 2014-07-21 13:52:33,950 Trac[PyGIT] DEBUG: git exits with 1, dir: > u'/home/git/repositories/boost-cgi.git', args: config ('--get', > 'i18n.commitEncoding'), stderr: '' > 2014-07-21 13:52:33,951 Trac[api] DEBUG: Event changeset_added on > repository 'boost-cgi' for revision > '86034d863b512607f374a0ebfd28f28f4f969662' > 2014-07-21 13:52:33,953 Trac[commit_updater] DEBUG: Updating ticket #67 > 2014-07-21 13:52:33,953 Trac[api] DEBUG: action controllers for ticket > workflow: ['ConfigurableTicketWorkflow'] > 2014-07-21 13:52:33,988 Trac[notification] INFO: Sending notification > through sendmail at /usr/sbin/sendmail to > [u'[email protected] <javascript:>', > u'[email protected] <javascript:>'] > 2014-07-21 13:52:33,988 Trac[notification] DEBUG: Sendmail command line: > [u'/usr/sbin/sendmail', '-i', '-f', u'[email protected] <javascript:>', > u'[email protected] <javascript:>', > u'[email protected] <javascript:>'] > > > [email protected] <javascript:> is the commiter (not registered anywhere > in Trac) and > [email protected] <javascript:> is owner of ticket #67 > > Regards, > > Sylvain Raybaud >
I can only make some guesses based on the source code: http://trac.edgewall.org/browser/trunk/tracopt/ticket/commit_updater.py?rev=12865&marks=270-274#L259 - Confirm that commit_ticket_update_check_perms is in the [ticket] section. The value defaults to true anyway, so it seems unlikely this is the issue. - Do you have any special permission policies? You could try adding some logging to line 270: self.log.info("check_perms = %s, checking TICKET_MODIFY user %s: %s", self.check_perms, authname, 'TICKET_MODIFY' in perm) -- You received this message because you are subscribed to the Google Groups "Trac Users" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To post to this group, send email to [email protected]. Visit this group at http://groups.google.com/group/trac-users. For more options, visit https://groups.google.com/d/optout.
