Browse Source

Expect an array or single package name in packages param

Jordi Boggiano 5 years ago
parent
commit
0716682035
1 changed files with 5 additions and 3 deletions
  1. 5 3
      src/Packagist/WebBundle/Controller/ApiController.php

+ 5 - 3
src/Packagist/WebBundle/Controller/ApiController.php

@@ -257,9 +257,11 @@ class ApiController extends Controller
      */
     public function securityAdvisoryAction(Request $request): JsonResponse
     {
-        $packageNames = array_values(array_filter(array_map(function (string $packageName) {
-            return trim($packageName);
-        }, explode(',', $request->get('packages')))));
+        $packageNames = array_filter((array) $request->get('packages'));
+        if (!$packageNames) {
+            return new JsonResponse(['status' => 'error', 'message' => 'Missing array of package names as the "packages" parameter'], 400);
+        }
+
         $updatedSince = $request->query->getInt('updatedSince', 0);
 
         /** @var array[] $advisories */