Bug 62297

Summary: Add support for multiple hosts/contexts in CrawlerSessionManagerValve
Product: Tomcat 9 Reporter: Felix Schumacher <felix.schumacher>
Component: CatalinaAssignee: Tomcat Developers Mailing List <dev>
Severity: minor    
Priority: P2    
Version: unspecified   
Target Milestone: -----   
Hardware: All   
OS: All   
Attachments: Support multiple hosts in one CrawlerSessionManagerValve

Description Felix Schumacher 2018-04-12 21:12:30 UTC
Created attachment 35867 [details]
Support multiple hosts in one CrawlerSessionManagerValve

When the Crawler valve is set up for multiple hosts like in
  <Host name="localhost" ... />
  <Host name="example.invalid" ... />
  <Valve className="org.apache.catalina.valves.CrawlerSessionManagerValve" />

it is only managing the sessions for "localhost" as it is using the client ip as the only key.

See https://lists.apache.org/thread.html/3eb73d69855586717fa29ec5527ca4efdf64b4465425bc8c38c8ff83@%3Cusers.tomcat.apache.org%3E for more information
Comment 1 Mark Thomas 2018-04-23 19:56:59 UTC
Fixed in:
- trunk for 9.0.8 onwards
- 8.5.x for 8.5.31 onwards
- 8.0.x for 8.0.52 onwards
- 7.0.x for 7.0.87 onwards