Skip to content

Commit

Permalink
Uses --end-of-options after command options (for security reasons)
Browse files Browse the repository at this point in the history
  • Loading branch information
janpecha committed Apr 21, 2022
1 parent b00d8dd commit 5e82d54
Show file tree
Hide file tree
Showing 6 changed files with 48 additions and 45 deletions.
3 changes: 3 additions & 0 deletions src/Git.php
Original file line number Diff line number Diff line change
Expand Up @@ -46,6 +46,7 @@ public function init($directory, array $params = NULL)
$this->run($directory, [
'init',
$params,
'--end-of-options',
$directory
]);

Expand Down Expand Up @@ -89,6 +90,7 @@ public function cloneRepository($url, $directory = NULL, array $params = NULL)
$this->run($cwd, [
'clone',
$params,
'--end-of-options',
$url,
$directory
]);
Expand Down Expand Up @@ -120,6 +122,7 @@ public function isRemoteUrlReadable($url, array $refs = NULL)
'--heads',
'--quiet',
'--exit-code',
'--end-of-options',
$url,
$refs,
], [
Expand Down
30 changes: 15 additions & 15 deletions src/GitRepository.php
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,7 @@ public function getRepositoryPath()
*/
public function createTag($name, $options = NULL)
{
$this->run('tag', $options, $name);
$this->run('tag', $options, '--end-of-options', $name);
return $this;
}

Expand Down Expand Up @@ -86,7 +86,7 @@ public function renameTag($oldName, $newName)
{
// http://stackoverflow.com/a/1873932
// create new as alias to old (`git tag NEW OLD`)
$this->run('tag', $newName, $oldName);
$this->run('tag', '--end-of-options', $newName, $oldName);
// delete old (`git tag -d OLD`)
$this->removeTag($oldName);
return $this;
Expand Down Expand Up @@ -114,7 +114,7 @@ public function getTags()
*/
public function merge($branch, $options = NULL)
{
$this->run('merge', $options, $branch);
$this->run('merge', $options, '--end-of-options', $branch);
return $this;
}

Expand All @@ -131,7 +131,7 @@ public function merge($branch, $options = NULL)
public function createBranch($name, $checkout = FALSE)
{
// git branch $name
$this->run('branch', $name);
$this->run('branch', '--end-of-options', $name);

if ($checkout) {
$this->checkout($name);
Expand Down Expand Up @@ -234,7 +234,7 @@ public function getLocalBranches()
*/
public function checkout($name)
{
$this->run('checkout', $name);
$this->run('checkout', '--end-of-options', $name);
return $this;
}

Expand All @@ -253,7 +253,7 @@ public function removeFile($file)
}

foreach ($file as $item) {
$this->run('rm', $item, '-r');
$this->run('rm', '-r', '--end-of-options', $item);
}

return $this;
Expand Down Expand Up @@ -282,7 +282,7 @@ public function addFile($file)
throw new GitException("The path at '$item' does not represent a valid file.");
}

$this->run('add', $item);
$this->run('add', '--end-of-options', $item);
}

return $this;
Expand Down Expand Up @@ -319,7 +319,7 @@ public function renameFile($file, $to = NULL)
}

foreach ($file as $from => $to) {
$this->run('mv', $from, $to);
$this->run('mv', '--end-of-options', $from, $to);
}

return $this;
Expand Down Expand Up @@ -454,7 +454,7 @@ public function hasChanges()
*/
public function pull($remote = NULL, array $params = NULL)
{
$this->run('pull', $remote, $params);
$this->run('pull', $params, '--end-of-options', $remote);
return $this;
}

Expand All @@ -468,7 +468,7 @@ public function pull($remote = NULL, array $params = NULL)
*/
public function push($remote = NULL, array $params = NULL)
{
$this->run('push', $remote, $params);
$this->run('push', $params, '--end-of-options', $remote);
return $this;
}

Expand All @@ -482,7 +482,7 @@ public function push($remote = NULL, array $params = NULL)
*/
public function fetch($remote = NULL, array $params = NULL)
{
$this->run('fetch', $remote, $params);
$this->run('fetch', $params, '--end-of-options', $remote);
return $this;
}

Expand All @@ -497,7 +497,7 @@ public function fetch($remote = NULL, array $params = NULL)
*/
public function addRemote($name, $url, array $params = NULL)
{
$this->run('remote', 'add', $params, $name, $url);
$this->run('remote', 'add', $params, '--end-of-options', $name, $url);
return $this;
}

Expand All @@ -511,7 +511,7 @@ public function addRemote($name, $url, array $params = NULL)
*/
public function renameRemote($oldName, $newName)
{
$this->run('remote', 'rename', $oldName, $newName);
$this->run('remote', 'rename', '--end-of-options', $oldName, $newName);
return $this;
}

Expand All @@ -524,7 +524,7 @@ public function renameRemote($oldName, $newName)
*/
public function removeRemote($name)
{
$this->run('remote', 'remove', $name);
$this->run('remote', 'remove', '--end-of-options', $name);
return $this;
}

Expand All @@ -539,7 +539,7 @@ public function removeRemote($name)
*/
public function setRemoteUrl($name, $url, array $params = NULL)
{
$this->run('remote', 'set-url', $params, $name, $url);
$this->run('remote', 'set-url', $params, '--end-of-options', $name, $url);
return $this;
}

Expand Down
10 changes: 5 additions & 5 deletions tests/GitPhp/GitRepository.branches.phpt
Original file line number Diff line number Diff line change
Expand Up @@ -10,12 +10,12 @@ require __DIR__ . '/bootstrap.php';
$runner = new AssertRunner(__DIR__);
$git = new Git($runner);

$runner->assert(['branch', 'master']);
$runner->assert(['branch', 'develop']);
$runner->assert(['checkout', 'develop']);
$runner->assert(['merge', 'feature-1']);
$runner->assert(['branch', '--end-of-options', 'master']);
$runner->assert(['branch', '--end-of-options', 'develop']);
$runner->assert(['checkout', '--end-of-options', 'develop']);
$runner->assert(['merge', '--end-of-options', 'feature-1']);
$runner->assert(['branch', '-d', 'feature-1']);
$runner->assert(['checkout', 'master']);
$runner->assert(['checkout', '--end-of-options', 'master']);

$repo = $git->open(__DIR__);
$repo->createBranch('master');
Expand Down
26 changes: 13 additions & 13 deletions tests/GitPhp/GitRepository.files.phpt
Original file line number Diff line number Diff line change
Expand Up @@ -14,11 +14,11 @@ $repo = $git->open(__DIR__ . '/fixtures');

test(function () use ($repo, $runner) {
$runner->resetAsserts();
$runner->assert(['add', 'file1.txt']);
$runner->assert(['add', 'file2.txt']);
$runner->assert(['add', 'file3.txt']);
$runner->assert(['add', 'file4.txt']);
$runner->assert(['add', 'file5.txt']);
$runner->assert(['add', '--end-of-options', 'file1.txt']);
$runner->assert(['add', '--end-of-options', 'file2.txt']);
$runner->assert(['add', '--end-of-options', 'file3.txt']);
$runner->assert(['add', '--end-of-options', 'file4.txt']);
$runner->assert(['add', '--end-of-options', 'file5.txt']);

$repo->addFile('file1.txt');
$repo->addFile([
Expand All @@ -38,11 +38,11 @@ test(function () use ($repo) {

test(function () use ($repo, $runner) {
$runner->resetAsserts();
$runner->assert(['rm', 'file1.txt', '-r']);
$runner->assert(['rm', 'file2.txt', '-r']);
$runner->assert(['rm', 'file3.txt', '-r']);
$runner->assert(['rm', 'file4.txt', '-r']);
$runner->assert(['rm', 'file5.txt', '-r']);
$runner->assert(['rm', '-r', '--end-of-options', 'file1.txt']);
$runner->assert(['rm', '-r', '--end-of-options', 'file2.txt']);
$runner->assert(['rm', '-r', '--end-of-options', 'file3.txt']);
$runner->assert(['rm', '-r', '--end-of-options', 'file4.txt']);
$runner->assert(['rm', '-r', '--end-of-options', 'file5.txt']);

$repo->removeFile('file1.txt');
$repo->removeFile([
Expand All @@ -55,9 +55,9 @@ test(function () use ($repo, $runner) {

test(function () use ($repo, $runner) {
$runner->resetAsserts();
$runner->assert(['mv', 'file1.txt', 'new1.txt']);
$runner->assert(['mv', 'file2.txt', 'new2.txt']);
$runner->assert(['mv', 'file3.txt', 'new3.txt']);
$runner->assert(['mv', '--end-of-options', 'file1.txt', 'new1.txt']);
$runner->assert(['mv', '--end-of-options', 'file2.txt', 'new2.txt']);
$runner->assert(['mv', '--end-of-options', 'file3.txt', 'new3.txt']);

$repo->renameFile('file1.txt', 'new1.txt');
$repo->renameFile([
Expand Down
20 changes: 10 additions & 10 deletions tests/GitPhp/GitRepository.remotes.phpt
Original file line number Diff line number Diff line change
Expand Up @@ -10,17 +10,17 @@ require __DIR__ . '/bootstrap.php';
$runner = new AssertRunner(__DIR__);
$git = new Git($runner);

$runner->assert(['clone', '-q', '[email protected]:czproject/git-php.git', __DIR__]);
$runner->assert(['remote', 'add', 'origin2', '[email protected]:czproject/git-php.git']);
$runner->assert(['remote', 'add', 'remote', '[email protected]:czproject/git-php.git']);
$runner->assert(['clone', '-q', '--end-of-options', '[email protected]:czproject/git-php.git', __DIR__]);
$runner->assert(['remote', 'add', '--end-of-options', 'origin2', '[email protected]:czproject/git-php.git']);
$runner->assert(['remote', 'add', '--end-of-options', 'remote', '[email protected]:czproject/git-php.git']);
$runner->assert(['remote', 'add', [
'--mirror=push',
], 'only-push', 'test-url']);
$runner->assert(['remote', 'rename', 'remote', 'origin3']);
], '--end-of-options', 'only-push', 'test-url']);
$runner->assert(['remote', 'rename', '--end-of-options', 'remote', 'origin3']);
$runner->assert(['remote', 'set-url', [
'--push',
], 'origin3', 'test-url']);
$runner->assert(['remote', 'remove', 'origin2']);
], '--end-of-options', 'origin3', 'test-url']);
$runner->assert(['remote', 'remove', '--end-of-options', 'origin2']);

$repo = $git->cloneRepository('[email protected]:czproject/git-php.git', __DIR__);
$repo->addRemote('origin2', '[email protected]:czproject/git-php.git');
Expand All @@ -34,9 +34,9 @@ $repo->setRemoteUrl('origin3', 'test-url', [
]);
$repo->removeRemote('origin2');

$runner->assert(['push', 'origin']);
$runner->assert(['fetch', 'origin']);
$runner->assert(['pull', 'origin']);
$runner->assert(['push', '--end-of-options', 'origin']);
$runner->assert(['fetch', '--end-of-options', 'origin']);
$runner->assert(['pull', '--end-of-options', 'origin']);
$repo->push('origin');
$repo->fetch('origin');
$repo->pull('origin');
4 changes: 2 additions & 2 deletions tests/GitPhp/GitRepository.tags.phpt
Original file line number Diff line number Diff line change
Expand Up @@ -10,8 +10,8 @@ require __DIR__ . '/bootstrap.php';
$runner = new AssertRunner(__DIR__);
$git = new Git($runner);

$runner->assert(['tag', 'v1.0.0']);
$runner->assert(['tag', 'v2.0.0', 'v1.0.0']);
$runner->assert(['tag', '--end-of-options', 'v1.0.0']);
$runner->assert(['tag', '--end-of-options', 'v2.0.0', 'v1.0.0']);
$runner->assert(['tag', '-d', 'v1.0.0']);
$runner->assert(['tag', '-d', 'v2.0.0']);

Expand Down

0 comments on commit 5e82d54

Please sign in to comment.