Skynet/misc_pterodactyl-panel
6
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix bug with permissions including more than one dash, closes #727

Browse source
This commit is contained in:
Dane Everitt 2017-11-05 13:42:57 -06:00
parent 30ab6ed692
commit 2f42396317
No known key found for this signature in database
GPG key ID: EEA66103B3D71F53
2 changed files with 8 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
app/Services/Api/KeyCreationService.php
View file

@ -86,7 +86,7 @@ class KeyCreationService
$nodes = $this->permissionService->getPermissions();
foreach ($permissions as $permission) {
@list($block, $search) = explode('-', $permission);
@list($block, $search) = explode('-', $permission, 2);
if (
(empty($block) || empty($search)) ||
@ -100,7 +100,7 @@ class KeyCreationService
}
foreach ($administrative as $permission) {
@list($block, $search) = explode('-', $permission);
@list($block, $search) = explode('-', $permission, 2);
if (
(empty($block) || empty($search)) ||

10
tests/Unit/Services/Api/KeyCreationServiceTest.php
View file

@ -82,19 +82,21 @@ class KeyCreationServiceTest extends TestCase
], true, true)->once()->andReturn((object) ['id' => 1]);
$this->permissions->shouldReceive('getPermissions')->withNoArgs()->once()->andReturn([
'_user' => ['server' => ['list']],
'server' => ['create'],
'_user' => ['server' => ['list', 'multiple-dash-test']],
'server' => ['create', 'admin-dash-test'],
]);
$this->permissions->shouldReceive('create')->with(1, 'user.server-list')->once()->andReturnNull();
$this->permissions->shouldReceive('create')->with(1, 'user.server-multiple-dash-test')->once()->andReturnNull();
$this->permissions->shouldReceive('create')->with(1, 'server-create')->once()->andReturnNull();
$this->permissions->shouldReceive('create')->with(1, 'server-admin-dash-test')->once()->andReturnNull();
$this->connection->shouldReceive('commit')->withNoArgs()->once()->andReturnNull();
$response = $this->service->handle(
['test-data' => 'test'],
['invalid-node', 'server-list'],
['invalid-node', 'server-create']
['invalid-node', 'server-list', 'server-multiple-dash-test'],
['invalid-node', 'server-create', 'server-admin-dash-test']
);
$this->assertNotEmpty($response);