Modify ↓
#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
197 197 keys = req.session.keys() 198 198 for key in keys: 199 199 if key.startswith('captcha_'): 200 self.log. warning("Remove useless key: "201 200 self.log.info("Remove useless key: " 201 "%s", key) 202 202 del req.session[key] 203 203 handler = newhandler 204 204 break
Attachments (0)
Change History (5)
comment:1 by , 9 years ago
comment:2 by , 9 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
Thank you for the fast reply. Committed in [14363].
comment:3 by , 9 years ago
Owner: | changed from | to
---|
comment:4 by , 9 years ago
Release Notes: | modified (diff) |
---|
comment:5 by , 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.
Note:
See TracTickets
for help on using tickets.
No objections.