diff --git a/src/Controller/GameController.php b/src/Controller/GameController.php index d307cf3..d178442 100644 --- a/src/Controller/GameController.php +++ b/src/Controller/GameController.php @@ -21,7 +21,6 @@ use Symfony\Component\DependencyInjection\Attribute\Autowire; use Symfony\Component\HttpFoundation\Request; use Symfony\Component\HttpFoundation\Response; use Symfony\Component\HttpKernel\Attribute\AsController; -use Symfony\Component\Mailer\MailerInterface; use Symfony\Component\Routing\Attribute\Route; /** @@ -85,8 +84,11 @@ class GameController extends AbstractController public function contact( Request $request, EntityManagerInterface $em, - MailerInterface $mailer, ): Response { + // Keep the contact form implementation available, but disable public + // access while automated abuse protection is insufficient. + throw $this->createNotFoundException(); + $contactMessage = new ContactMessage(); $form = $this->createForm(ContactFormType::class, $contactMessage); $form->handleRequest($request); diff --git a/templates/Game/index.html.twig b/templates/Game/index.html.twig index b9b5fbc..55c2b30 100644 --- a/templates/Game/index.html.twig +++ b/templates/Game/index.html.twig @@ -260,7 +260,6 @@
  • Game Rules
  • Terms of Use
  • Privacy Policy
  • -
  • Contact
  • diff --git a/tests/Controller/GameControllerTest.php b/tests/Controller/GameControllerTest.php index a3005b7..51d3796 100644 --- a/tests/Controller/GameControllerTest.php +++ b/tests/Controller/GameControllerTest.php @@ -89,14 +89,12 @@ class GameControllerTest extends WebTestCase } #[Test] - #[TestDox('Contact page loads successfully with form')] - public function contactPageLoadsSuccessfully(): void + #[TestDox('Contact page is not publicly accessible')] + public function contactPageIsNotPubliclyAccessible(): void { $crawler = $this->client->request('GET', '/contact'); - self::assertResponseStatusCodeSame(Response::HTTP_OK); - self::assertSelectorExists('form'); - self::assertSelectorExists('button[type="submit"]'); + self::assertResponseStatusCodeSame(Response::HTTP_NOT_FOUND); } #[Test]