Headers check should be lowercase

Created on 18 December 2024, 2 months ago

Problem/Motivation

In Acquia hosting environments, and potentially others as well, the getHeaders() method returns $headers all in lowercase. This is opposed to other environments which returns them in sentence case. This is causing the "All specified headers present check" to fail even when the headers are present.

Steps to reproduce

1. Have an Acquia hosted site.
2. Verify that X-Frame-Options header is present and set to SAMEORIGIN
3. Run the security review check.
4. The check fails.

Proposed resolution

Make the check case-insensitive.

πŸ› Bug report
Status

Active

Version

3.1

Component

Code

Created by

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

Production build 0.71.5 2024