Bug 57705 - RemoteAddrValve: no log message no explanation when valve rejects request
Summary: RemoteAddrValve: no log message no explanation when valve rejects request
Status: RESOLVED FIXED
Alias: None
Product: Tomcat 7
Classification: Unclassified
Component: Catalina (show other bugs)
Version: 7.0.59
Hardware: PC Mac OS X 10.1
: P2 enhancement (vote)
Target Milestone: ---
Assignee: Tomcat Developers Mailing List
URL:
Keywords: PatchAvailable
Depends on:
Blocks:
 
Reported: 2015-03-14 13:35 UTC by Graham Leggett
Modified: 2016-06-16 13:04 UTC (History)
0 users



Attachments
Add debug log when request is denied (1.17 KB, patch)
2015-03-15 12:16 UTC, Graham Leggett
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Graham Leggett 2015-03-14 13:35:04 UTC
When the RemoteAddrValve denies a request, no message is logged, and no explanation appears in the generated HTML page.

There needs to be some indication that the RemoteAddrValve is involved so the admin isn't sent on a wild goose chase.
Comment 1 Mark Thomas 2015-03-14 14:26:47 UTC
Patches welcome.
Comment 2 Graham Leggett 2015-03-15 12:16:58 UTC
Created attachment 32570 [details]
Add debug log when request is denied

Patch added as requested, should this message be at debug level or at notice level?

A rejected request is usually the kind of thing you'd expect to be logged visibly.
Comment 3 Mark Thomas 2016-06-16 13:04:44 UTC
Thanks for the patch.

I applied a variantion that modified both the Valves and the Filters.

The new debug logging has been implemnted in:
- 9.0.x for 9.0.0.M9 onwards
- 8.5.x for 8.5.4 onwards
- 8.0.x for 8.0.37 onwards
- 7.0.x for 7.0.71 onwards