Skip to content

Commit 5871eef

Browse files
committed
bug #6902 Fix double encoded urls (ksn135)
This PR was merged into the 4.x branch. Discussion ---------- Fix double encoded urls Sometimes Telegram for iOS mistakenly double convert url if it contains unicode letters (issue overtake/TelegramSwift/issues/1215) Valid: https://example.com/?crudAction=detail&crudControllerFqcn=App%5CController%5CDocCrudController&entityId=2500426&page=1&query=гаврилов Broken (until this fix): https://example.com/?crudAction=detail&crudControllerFqcn=App%255CController%255CDocCrudController&entityId=2500426&page=1&query=гаврилов <!-- Thanks for your contribution! If you are proposing a new feature that is complex, please open an issue first so we can discuss about it. Note: all your contributions adhere implicitly to the MIT license --> Commits ------- b3f7119 Fix double encoded urls
2 parents a8bef2e + b3f7119 commit 5871eef

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

src/Factory/ControllerFactory.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -50,6 +50,7 @@ private function getController(string $controllerInterface, ?string $controllerF
5050
return null;
5151
}
5252

53+
$controllerFqcn = str_replace('%5C', '\\', $controllerFqcn);
5354
$newRequest = $request->duplicate(null, null, ['_controller' => [$controllerFqcn, $controllerAction]]);
5455
try {
5556
$controllerCallable = $this->controllerResolver->getController($newRequest);

0 commit comments

Comments
 (0)