Generated URLs should only have cache contexts on GET/HEAD

Created on 8 June 2016, almost 9 years ago
Updated 29 May 2025, 7 days ago

Problem/Motivation

Same thing as #2729211: node_query_node_access_alter() should only add cacheability on GET/HEAD .

In this case a URL was being generated for logging, but toString() was called without setting $collect_bubblable_metadata to FALSE, resulting in our favourite exception message.

Proposed resolution

Only collect bubbleable metadata if both the argument is set, and the method is safe. This is a behaviour change of course.

Remaining tasks

User interface changes

API changes

Data model changes

🐛 Bug report
Status

Needs work

Version

11.0 🔥

Component

base system

Created by

🇬🇧United Kingdom catch

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

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Production build 0.71.5 2024