Consider throwing BadRequestException in AjaxPageStateNegotiator

Created on 23 October 2023, over 1 year ago

Problem/Motivation

Prior to πŸ“Œ Compress ajax_page_state Fixed AjaxPageStateNegotiator checked for both theme and theme_token in its :applies method.

Now it doesn't check for the theme_token because this can be unset as well as null (since it may come from serialized JSON).

The behavior preserved in that issue was to silently not return anything if there's a non-default theme requested without any theme token. However, we could consider throwing BadRequestException and present a 400 to the client instead.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

πŸ“Œ Task
Status

Active

Version

11.0 πŸ”₯

Component
AjaxΒ  β†’

Last updated 43 minutes ago

Created by

πŸ‡¬πŸ‡§United Kingdom catch

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

Comments & Activities

Production build 0.71.5 2024