Skip to content

Commit 39f6727

Browse files
committed
Fixes #6
1 parent 3956dad commit 39f6727

File tree

2 files changed

+13
-5
lines changed

2 files changed

+13
-5
lines changed

src/Service/Provider/PackagesProvider.php

Lines changed: 11 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -142,18 +142,24 @@ private function getLocalRecipe(array $package)
142142
*
143143
* @param string $packagesRequestString
144144
* @return array
145+
* @throws \HttpRequestException
145146
*/
146147
private function parseRequestedPackages(string $packagesRequestString)
147148
{
148149
$packages = [];
149150
foreach (explode(';', rtrim($packagesRequestString, ';')) as $requestedPackage) {
150-
[$author, $package, $version, $timestamp] = explode(',', $requestedPackage);
151+
$packageDetails = explode(',', $requestedPackage);
152+
153+
if (count($packageDetails) < 3) {
154+
throw new \HttpRequestException('Invalid package string provided', 500);
155+
}
156+
151157
$packages[] = [
152-
'author' => $author,
153-
'package' => $package,
154-
'version' => preg_replace('/^[iurv]+/', '', $version)
158+
'author' => $packageDetails[0],
159+
'package' => $packageDetails[1],
160+
'version' => preg_replace('/^[iurv]+/', '', $packageDetails[2])
155161
];
156162
}
157163
return $packages;
158164
}
159-
}
165+
}

tests/repo/.gitignore

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,2 @@
1+
*
2+
!.gitignore

0 commit comments

Comments
 (0)