소스 검색

Minor fixes/tweaks

Jordi Boggiano 5 년 전
부모
커밋
0030b23817
2개의 변경된 파일3개의 추가작업 그리고 3개의 파일을 삭제
  1. 2 2
      src/Packagist/WebBundle/Controller/WebController.php
  2. 1 1
      src/Packagist/WebBundle/HealthCheck/RedisHealthCheck.php

+ 2 - 2
src/Packagist/WebBundle/Controller/WebController.php

@@ -116,7 +116,7 @@ class WebController extends Controller
             $query = $form->getData()->getQuery();
         }
 
-        $perPage = $req->query->getInt('per_page', 15);
+        $perPage = max(1, (int) $req->query->getInt('per_page', 15));
         if ($perPage <= 0 || $perPage > 100) {
            if ($req->getRequestFormat() === 'json') {
                 return JsonResponse::create(array(
@@ -132,7 +132,7 @@ class WebController extends Controller
             $queryParams['filters'] = implode(' AND ', $queryParams['filters']);
         }
         $queryParams['hitsPerPage'] = $perPage;
-        $queryParams['page'] = $req->query->get('page', 1) - 1;
+        $queryParams['page'] = max(1, (int) $req->query->get('page', 1)) - 1;
 
         try {
             $results = $index->search($query, $queryParams);

+ 1 - 1
src/Packagist/WebBundle/HealthCheck/RedisHealthCheck.php

@@ -27,7 +27,7 @@ class RedisHealthCheck extends AbstractCheck
             }
 
             // only warn for fragmented memory when amount of memory is above 256MB
-            if ($info['Memory']['used_memory'] > 256*1024*1024 && $info['Memory']['mem_fragmentation_ratio'] > 1.6) {
+            if ($info['Memory']['used_memory'] > 256*1024*1024 && $info['Memory']['mem_fragmentation_ratio'] > 3) {
                 return new Warning('Redis memory fragmentation ratio is pretty high, maybe redis instances should be restarted', $info['Memory']);
             }