Skip to content

Comments

Prevent a urlencoded sub-URL in main URL path to be decoded.#95

Merged
LukeTowers merged 5 commits intowip/1.2from
test-urlgenerator
Jul 13, 2022
Merged

Prevent a urlencoded sub-URL in main URL path to be decoded.#95
LukeTowers merged 5 commits intowip/1.2from
test-urlgenerator

Conversation

@mjauvin
Copy link
Member

@mjauvin mjauvin commented Jul 10, 2022

Fixes changes done in 32df10c

The above commit broke the ImageResizer encoded URL.

@mjauvin mjauvin added this to the v1.2.0 milestone Jul 10, 2022
@mjauvin mjauvin requested a review from LukeTowers July 10, 2022 17:48
@mjauvin mjauvin self-assigned this Jul 10, 2022
@mjauvin mjauvin changed the title Add unit test with doubly encoded url in URL path Prevent a urlencoded sub-URL in main URL path to be decoded. Jul 13, 2022
@mjauvin mjauvin requested a review from bennothommo July 13, 2022 11:10
@LukeTowers LukeTowers merged commit 768c2c4 into wip/1.2 Jul 13, 2022
@LukeTowers LukeTowers deleted the test-urlgenerator branch July 13, 2022 20:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants