Problem/Motivation

During tested we figured that drupal cache is not respecting the max-age set in https://git.drupalcode.org/project/ohdear_integration/-/blob/2.x/src/Con... and responses are cached much longer than they should be.

Also CDN provider notified us that Last-Modified or Etag headers are missing.

Acceptance criteria

Make sure that drupal cache respects the healthcheck_cache_max_age configuration and add appropriate headers to response.

Issue fork ohdear_integration-3450841

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

useernamee created an issue. See original summary.

useernamee’s picture

useernamee commented
Priority: Normal » Critical
useernamee’s picture

useernamee commented
Status: Active » Needs review

During working on this ticket I noticed that drupal cache gets permanent status if expires header is not set, that's why added that one as well.

useernamee opened merge request !23

petar_basic’s picture

petar_basic commented
Status: Needs review » Reviewed & tested by the community
useernamee’s picture

useernamee commented

  • useernamee committed f45b367c on 2.x 
    Issue #3450841 by useernamee, petar_basic: Caching issue
useernamee’s picture

useernamee commented
Status: Reviewed & tested by the community » Fixed

merged

creating a new release - 2.0.2

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.