summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Newline after "let"key-validationsJacob Vosmaer2016-08-101-0/+1
|
* Changes proposed during reviewJacob Vosmaer2016-08-102-3/+3
|
* Defense in depth for authorized_keys linesJacob Vosmaer2016-08-052-0/+31
|
* Merge branch 'small-fixes' into 'master' Robert Speicher2016-08-043-17/+19
|\ | | | | | | | | | | | | | | Small improvements - refactor GitlabKey away - fix Redis check output See merge request !81
| * Refactor 'GitlabKey' class awaysmall-fixesJacob Vosmaer2016-08-042-16/+12
| | | | | | | | | | It is not nice to have both 'GitlabKeys' and 'GitlabKey'. We also do not need GitlabKey to be a class when it has no state.
| * Improve output from redis checkJacob Vosmaer2016-08-041-1/+7
|/
* Bump version to 3.3.2v3.3.2Douwe Maan2016-08-032-1/+4
|
* Merge branch 'authorized-keys-permission-check' into 'master' Douwe Maan2016-08-033-6/+36
|\ | | | | | | | | | | | | | | | | Improve authorized_keys check The old check only looked if authorized_keys exists. With this change, we look whether we can actually open the file for reading and writing. When this fails we try to print useful diagnostic information. See merge request !79
| * Add permission check tests, improve checkauthorized-keys-permission-checkJacob Vosmaer2016-08-032-4/+23
| |
| * Improve authorized_keys checkJacob Vosmaer2016-08-022-5/+16
|/ | | | | | The old check only looked if authorized_keys exists. With this change, we look whether we can actually open the file for reading and writing. When this fails we try to print useful diagnostic information.
* Bump version to 3.3.1v3.3.1Douwe Maan2016-08-012-1/+4
|
* Merge branch 'keys-chmod' into 'master' Douwe Maan2016-08-013-7/+12
|\ | | | | | | | | | | | | | | | | Manage authorized_keys permissions continuously We can lazily create authorized_keys and set its permissions. This adds negligible overhead and it allows us to remove a setup step from GitLab both on source and in omnibus-gitlab. See merge request !78
| * Manage authorized_keys permissions continuouslykeys-chmodJacob Vosmaer2016-08-013-7/+12
|/ | | | | | We can lazily create authorized_keys and set its permissions. This adds negligible overhead and it allows us to remove a setup step from GitLab both on source and in omnibus-gitlab.
* Version 3.3.0v3.3.0Jacob Vosmaer2016-07-271-1/+1
|
* Merge branch 'mv-storage' into 'master' Jacob Vosmaer (GitLab)2016-07-278-2/+232
|\ | | | | | | | | | | | | | | | | Add command to move repositories between repository storages Necessary for https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/533 /cc @pcarranza See merge request !71
| * Track ongoing pushes and reject mv-storage commands if there are push ↵mv-storageAlejandro Rodríguez2016-07-278-4/+162
| | | | | | | | running (after waiting some time)
| * Add command to move repositories between repository storagesAlejandro Rodríguez2016-07-273-0/+72
|/
* Bump version to 3.2.1v3.2.1Douwe Maan2016-07-261-1/+1
|
* Merge branch ↵Douwe Maan2016-07-263-8/+43
|\ | | | | | | | | | | | | | | | | | | '51-gitlab-project-s-fork_project-command-doesn-t-work-if-the-project-is-being-forked-to-another-storage' into 'master' Allow gitlab-project's fork-project command to fork projects between different repository storages Closes #51 See merge request !75
| * Allow gitlab-project's fork-project command to fork projects between ↵Alejandro Rodríguez2016-07-263-8/+43
|/ | | | different repository storages
* Merge branch 'remove-env-set' into 'master' Robert Speicher2016-07-213-10/+4
|\ | | | | | | | | | | | | | | | | Unset GL_ID and GL_PROTOCOL immediately Neither other code executing in the hook process nor subprocesses of the hook process need access to the GL_ID / GL_PROTOCOL variables so we can just unset them the moment we fetch them. See merge request !73
| * Remove spaceremove-env-setJacob Vosmaer2016-07-201-1/+1
| |
| * More ENV.deleteJacob Vosmaer2016-07-201-1/+1
| |
| * Unset GL_ID and GL_PROTOCOL immediatelyJacob Vosmaer2016-07-202-9/+3
|/ | | | | | Neither other code executing in the hook process nor subprocesses of the hook process need access to the GL_ID / GL_PROTOCOL variables so we can just unset them the moment we fetch them.
* Merge branch '48-regression-in-v3-1-0-vs-v3-0-0' into 'master' v3.2.0Stan Hu2016-07-062-0/+2
|\ | | | | | | | | | | | | Add an error message when using shell commands with incompatible GitLab versions Closes #48 See merge request !70
| * Add an error message when using shell commands with incompatible GitLab versionsAlejandro Rodríguez2016-07-062-0/+2
|/
* Merge branch 'add-allowed-protocols-support' into 'master' Douwe Maan2016-07-0611-17/+150
|\ | | | | | | | | | | | | Allow GitLab Shell to check for allowed access based on the used Git protocol. Needed for gitlab-org/gitlab-ce!4696 and gitlab-org/gitlab-ce#18601 See merge request !62
| * Bump VERSION to 3.2.0add-allowed-protocols-supportPatricio Cano2016-07-052-2/+4
| |
| * Make use of a constant for the used SSH protocolPatricio Cano2016-07-051-2/+3
| |
| * Rename ENV['PROTOCOL'] to ENV['GL_PROTOCOL'] and make it mandatory with no ↵Patricio Cano2016-07-055-11/+11
| | | | | | | | fallback value
| * Added better tests for the protocol checkPatricio Cano2016-07-054-1/+126
| |
| * Simplify protocol assign, and populate ENV['PROTOCOL'] variable when calling ↵Patricio Cano2016-07-053-5/+4
| | | | | | | | hooks via SSH
| * Added CHANGELOG itemPatricio Cano2016-07-051-0/+1
| |
| * Allow GitLab Shell to check for allowed access based on the used Git protocol.Patricio Cano2016-07-054-6/+11
|/
* Merge branch 'remove-hound-config' into 'master' Stan Hu2016-07-051-3/+0
|\ | | | | | | | | | | | | Remove hound config Since we moved to gitlab.com we don't use Hound CI anymore. Instead we have rubocop. See merge request !69
| * Remove hound configremove-hound-configRobert Schilling2016-07-051-3/+0
|/
* Merge branch 'user_stderr' into 'master' Stan Hu2016-07-051-3/+3
|\ | | | | | | | | | | | | | | | | Use STDERR to output error messages It's pretty minor change so I don't think we need a changelog item for this. Fixes https://gitlab.com/gitlab-org/gitlab-shell/issues/47 See merge request !68
| * use STDERR to output error messagesValery Sizov2016-07-051-3/+3
|/
* Merge branch 'shards' into 'master' v3.1.0Yorick Peterse2016-06-3017-150/+172
|\ | | | | | | | | | | | | Implement shard paths handling See https://gitlab.com/gitlab-org/gitlab-ee/issues/583 See merge request !61
| * Refactor repository paths handling to allow multiple git mount pointsshardsAlejandro Rodríguez2016-06-2917-150/+172
|/
* Merge branch 'custom-hook-output' into 'master' Stan Hu2016-06-261-29/+16
|\ | | | | | | | | | | | | | | | | Prevent blocking custom hook output streams Based on https://gitlab.com/gitlab-org/gitlab-shell/merge_requests/49 Fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/1458 See merge request !67
| * Simplify custom hook output handlingcustom-hook-outputJacob Vosmaer2016-06-241-29/+16
|/
* Merge branch 'gitlab-redis' into 'master' Stan Hu2016-06-233-2/+3
|\ | | | | | | | | | | | | Put Redis loading code in one place Before this change you could not run the gitlab_net specs in isolation. See merge request !64
| * Put Redis loading code in one placegitlab-redisJacob Vosmaer2016-06-233-2/+3
| |
* | Merge branch 'no-argf' into 'master' Robert Speicher2016-06-232-2/+2
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ARGF does more than we want ARGF is a virtual IO stream that treats the elements of ARGV as paths to files to be opened. In practice, git-receive-pack will probably never pass arguments to pre-receive and post-receive because it is not supposed to (according to 'man 5 githooks'). In case it does, however our hooks should not have the surprise behavior of treating those arguments as files to be opened. See merge request !66
| * | ARGF does more than we wantJacob Vosmaer2016-06-232-2/+2
|/ / | | | | | | | | | | | | | | | | ARGF is a virtual IO stream that treats the elements of ARGV as paths to files to be opened. In practice, git-receive-pack will probably never pass arguments to pre-receive and post-receive because it is not supposed to (according to 'man 5 githooks'). In case it does, however our hooks should not have the surprise behavior of treating those arguments as files to be opened.
* | Merge branch 'simplify-access-status' into 'master' Valery Sizov2016-06-233-9/+8
|\ \ | | | | | | | | | | | | | | | | | | | | | Simplify the GitAccessStatus class Make the optional 'message' argument required. Remove unused 'to_json' method. See merge request !65
| * | Simplify the GitAccessStatus classsimplify-access-statusJacob Vosmaer2016-06-233-9/+8
| |/ | | | | | | | | Make the optional 'message' argument required. Remove unused 'to_json' method.
* | Merge branch 'enqueued_at_for_jobs' into 'master' Yorick Peterse2016-06-233-2/+12
|\ \ | |/ |/| | | | | Update PostReceive worker to provide enqueued_at time See merge request !63
| * Update PostReceive worker to provide enqueued_at timeenqueued_at_for_jobsPaco Guzman2016-06-233-2/+12
|/
View Lorry Controller Status