Skip to content

Commit e9540d0

Browse files
committed
Add caching for providers route
1 parent 88f61aa commit e9540d0

File tree

1 file changed

+10
-2
lines changed

1 file changed

+10
-2
lines changed

src/Controller/PackageController.php

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -353,7 +353,11 @@ public function viewProvidersAction(Request $req, string $name, RedisClient $red
353353
$providers = $repo->findProviders($name);
354354
if (!$providers) {
355355
if ($req->getRequestFormat() === 'json') {
356-
return new JsonResponse(['providers' => []]);
356+
$response = new JsonResponse(['providers' => []]);
357+
$response->setSharedMaxAge(300);
358+
$response->headers->set(AbstractSessionListener::NO_AUTO_CACHE_CONTROL_HEADER, 'true');
359+
360+
return $response;
357361
}
358362

359363
return $this->redirect($this->generateUrl('search_web', ['q' => $name, 'reason' => 'package_not_found']));
@@ -392,7 +396,11 @@ public function viewProvidersAction(Request $req, string $name, RedisClient $red
392396
];
393397
}
394398

395-
return new JsonResponse(['providers' => $response]);
399+
$response = new JsonResponse(['providers' => $response]);
400+
$response->setSharedMaxAge(300);
401+
$response->headers->set(AbstractSessionListener::NO_AUTO_CACHE_CONTROL_HEADER, 'true');
402+
403+
return $response;
396404
}
397405

398406
return $this->render('package/providers.html.twig', [

0 commit comments

Comments
 (0)