Limit page size to prevent int overflow

Tobion · Tobion · commit 8e14d5c73e01 · 2022-12-30T01:53:46.000+01:00
diff --git a/src/Controller/BlogController.php b/src/Controller/BlogController.php
@@ -46,7 +46,7 @@ class BlogController extends AbstractController
      */
     #[Route('/', defaults: ['page' => '1', '_format' => 'html'], methods: ['GET'], name: 'blog_index')]
     #[Route('/rss.xml', defaults: ['page' => '1', '_format' => 'xml'], methods: ['GET'], name: 'blog_rss')]
-    #[Route('/page/{page<[1-9]\d*>}', defaults: ['_format' => 'html'], methods: ['GET'], name: 'blog_index_paginated')]
+    #[Route('/page/{page<[1-9]\d{0,8}>}', defaults: ['_format' => 'html'], methods: ['GET'], name: 'blog_index_paginated')]
     #[Cache(smaxage: 10)]
     public function index(Request $request, int $page, string $_format, PostRepository $posts, TagRepository $tags): Response
     {

Original file line number	Diff line number	Diff line change
`@@ -46,7 +46,7 @@ class BlogController extends AbstractController`
`46`	`46`	`*/`
`47`	`47`	`#[Route('/', defaults: ['page' => '1', '_format' => 'html'], methods: ['GET'], name: 'blog_index')]`
`48`	`48`	`#[Route('/rss.xml', defaults: ['page' => '1', '_format' => 'xml'], methods: ['GET'], name: 'blog_rss')]`
`49`		`- #[Route('/page/{page<[1-9]\d*>}', defaults: ['_format' => 'html'], methods: ['GET'], name: 'blog_index_paginated')]`
	`49`	`+ #[Route('/page/{page<[1-9]\d{0,8}>}', defaults: ['_format' => 'html'], methods: ['GET'], name: 'blog_index_paginated')]`
`50`	`50`	`#[Cache(smaxage: 10)]`
`51`	`51`	`public function index(Request $request, int $page, string $_format, PostRepository $posts, TagRepository $tags): Response`
`52`	`52`	`{`