[MAGNOLIA-5717] UnicodeNormalizationRequestWrapper uses URLDecoder for headers but UnicodeNormalizer for parameters Created: 17/Mar/14  Updated: 09/Jul/14  Resolved: 24/Jun/14

Status: Closed
Project: Magnolia
Component/s: core
Affects Version/s: 5.2.3
Fix Version/s: 5.2.6

Type: Bug Priority: Major
Reporter: Philip Mundt Assignee: Philip Mundt
Resolution: Fixed Votes: 0
Labels: quickwin
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Attachments: File UnicodeNormalizationRequestWrapper.patch    
Issue Links:
causality
caused by MAGNOLIA-3233 error on activate recursively a conte... Closed
is causing MGNLACTIVATION-88 Activation breaks when magnolia.utf8.... Closed
is causing MGNLREST-54 REST request with valid accept header... Closed
Template:
Acceptance criteria:
Empty
Task DoD:
[ ]* Doc/release notes changes? Comment present?
[ ]* Downstream builds green?
[ ]* Solution information and context easily available?
[ ]* Tests
[ ]* FixVersion filled and not yet released
[ ]  Architecture Decision Record (ADR)
Bug DoR:
[ ]* Steps to reproduce, expected, and actual results filled
[ ]* Affected version filled
Testcase included:
Yes

 Description   

UnicodeNormalizationRequestWrapper uses URLDecoder for headers but UnicodeNormalizer for parameters. The decoding of the headers causes issues, it will e.g. result in following Accept header:

application/xhtml+xmlapplication/xhtml xml

This behavior was introduced in MAGNOLIA-3233.

Attached is a patch for info.magnolia.cms.filters.UnicodeNormalizationRequestWrapperTest showing the malfunction.



 Comments   
Comment by Philip Mundt [ 11/Jun/14 ]

Attaching a patch for a fix (including test). Will remove the patch with just the test.

Generated at Mon Feb 12 04:07:43 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.