GeoServer

Caching headers issues

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 1.7.0-beta1
  • Fix Version/s: 1.6.5, 1.7.0-beta2
  • Component/s: WMS
  • Labels:
    None
  • Number of attachments :
    0

Description

Enabling caching for topp:states with an cache time of 1000 results in the following header set:

curl -I "http://localhost:8080/geoserver/wms?SRS=EPSG%3A4326&WIDTH=800&STYLES=&HEIGHT=317&LAYERS=topp%3Astates&FORMAT=image%2Fpng&SERVICE=WMS&VERSION=1.1.1&REQUEST=GetMap&EXCEPTIONS=application%2Fvnd.ogc.se_inimage&BBOX=-145.48948729687498,17.49445597548828,-46.211783703124986,56.83324602451172"
HTTP/1.1 200 OK
Expires: Thu, 01 Jan 1970 00:00:00 GMT
Set-Cookie: JSESSIONID=194q98cv8mgap;Path=/geoserver
Content-Type: application/xml
Content-Length: 501
Server: Jetty(6.1.8)

The expires field is wrong, the max-age in this case is missing (the user reported it being there, so maybe this is an extra problem in 1.7.x)

Issue Links

Activity

Hide
David G. Smith added a comment -

max-age appears to be working, would be good to have the 'expires' field able to be set to something more user-controllable, as opposed to already-expired (1970).

Show
David G. Smith added a comment - max-age appears to be working, would be good to have the 'expires' field able to be set to something more user-controllable, as opposed to already-expired (1970).
Hide
Andrea Aime added a comment -

The expires header is created automatically by Jetty when a session is being created, it's not GeoServer's direct doing. That said, the session was created for each request due to a misbehaviour in a filter that I've fixed (see related issue) so now the expires header is no more added. This should be enough for service requests, which are by design not session oriented.

Show
Andrea Aime added a comment - The expires header is created automatically by Jetty when a session is being created, it's not GeoServer's direct doing. That said, the session was created for each request due to a misbehaviour in a filter that I've fixed (see related issue) so now the expires header is no more added. This should be enough for service requests, which are by design not session oriented.

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: