Modify

Opened 21 months ago

Closed 18 months ago

Last modified 17 months ago

#10666 closed defect (fixed)

Error logging typo

Reported by: myingling Owned by: sandinak
Priority: normal Component: DirectoryAuthPlugin
Severity: normal Keywords:
Cc: Trac Release: 1.0

Description (last modified by rjollos)

Line 116 (r12025) of auth.py prints a log message, but has a typo since it only provides one string, but indicates two will print.

self.log.debug('Unable to find user %s listed in group: %s' % str(m))

This looks like a trivial bug, though I'm not clearly seeing which variable contains the user name. In the meantime I can remove a '%s' to make the bug go away.

A similar bug (new ticket?) shows up at lines 500-509. Data is stored to variable 'res' or an exception is thrown which prints a long message. Then execution can continue, but even if the exception was thrown, the variable is still accessed at 506.

The end results of each of these issues is that that Trac loads with an error, but the error looks like a bug in the plugin, not a configuration issue (which becomes visible when you dig through the logs).

Attachments (0)

Change History (2)

comment:1 Changed 18 months ago by anonymous

  • Resolution set to fixed
  • Status changed from new to closed

(In [12718]) Patched for the typo.

comment:2 Changed 17 months ago by rjollos

  • Description modified (diff)

Add Comment

Modify Ticket

Action
as closed .
The resolution will be deleted. Next status will be 'reopened'.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.