Edgewall Software
Modify

Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#12247 closed defect (fixed)

Suppress "Remove useless key" warnings

Reported by: Ryan J Ollos Owned by: Ryan J Ollos
Priority: normal Milestone: plugin - spam-filter
Component: plugin/spamfilter Version:
Severity: normal Keywords:
Cc: Branch:
Release Notes:

Log "Remove useless key" at INFO level rather than WARNING level.

API Changes:
Internal Changes:

Description

When I review the logs each day I see many instances of the following:

2015-11-10 09:43:57,765 Trac[api] WARNING: Remove useless key: captcha_reject_count
2015-11-10 09:43:57,766 Trac[api] WARNING: Remove useless key: captcha_expected

Is there any value to logging a warning here? If not, maybe we could change it to log at INFO level:

  • tracspamfilter/captcha/api.py

     
    197197                                keys = req.session.keys()
    198198                                for key in keys:
    199199                                    if key.startswith('captcha_'):
    200                                         self.log.warning("Remove useless key: "
    201                                                          "%s", key)
     200                                        self.log.info("Remove useless key: "
     201                                                      "%s", key)
    202202                                        del req.session[key]
    203203                                handler = newhandler
    204204                                break

Attachments (0)

Change History (5)

comment:1 by Dirk Stöcker, 9 years ago

No objections.

comment:2 by Ryan J Ollos, 9 years ago

Resolution: fixed
Status: newclosed

Thank you for the fast reply. Committed in [14363].

comment:3 by Ryan J Ollos, 9 years ago

Owner: changed from Dirk Stöcker to Ryan J Ollos

comment:4 by Ryan J Ollos, 9 years ago

Release Notes: modified (diff)

comment:5 by Ryan J Ollos, 9 years ago

Clarified warnings logged from FSpamList in [14841]. After the changes, the following is seen in the logs:

2016-06-14 10:13:46,633 Trac[fspamlist] WARNING: Error parsing response from FSpamList: syntax error: line 1, column 0
Response:
MySQL connection error. Please try again later or contact the administrator.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Ryan J Ollos.
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from Ryan J Ollos to the specified user.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.