Merge pull request #3034 from MTES-MCT/bugfix/3010-fix-show-uploaded-…

…file-not-found [QA - sentry] NotFoundHttpException sur les exports pdf des signalements
MTES-MCT · Sep 16, 2024 · 55eabdc · 55eabdc
2 parents fa56adf + b9aa732
commit 55eabdc
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/Controller/Security/SecurityController.php b/src/Controller/Security/SecurityController.php
@@ -42,7 +42,7 @@ public function showUploadedFile(
  $request = Request::createFromGlobals();
 
  if (!$this->isCsrfTokenValid('suivi_signalement_ext_file_view', $request->get('t')) && !$this->isGranted('SIGN_VIEW', $signalement)) {
- throw $this->createNotFoundException();
+ throw $this->createAccessDeniedException();
  }
  try {
  $variant = $request->query->get('variant');

diff --git a/tests/Functional/Controller/SecurityControllerTest.php b/tests/Functional/Controller/SecurityControllerTest.php
@@ -46,6 +46,6 @@ public function testShowUploadedWithInvalidToken(): void
  $client = static::createClient();
  $client->request('GET', '/_up/check.png');
 
- $this->assertResponseStatusCodeSame(404);
+ $this->assertResponseRedirects('/connexion');
  }
 }