summaryrefslogtreecommitdiff
path: root/lib
Commit message (Collapse)AuthorAgeFilesLines
...
| * typo in message: "accesible" -> "accessible"Christian González2015-01-011-1/+1
| |
* | masking access_token from url for loggingValery Sizov2015-01-131-0/+1
|/
* Merge pull request #197 from dblessing/fix/custom_hook_outputJacob Vosmaer2014-11-261-3/+3
|\ | | | | Fix custom hook output
| * Fix custom hook output and return valuesDrew Blessing2014-11-181-3/+3
| |
* | Add missing 'require_relative' in gitlab_net.rbJacob Vosmaer2014-11-251-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Without this 'require' statement, you get the following error: ``` $ git ls-remote origin /home/git/gitlab-shell/lib/gitlab_net.rb:30:in `check_access': uninitialized constant GitlabNet::GitAccessStatus (NameError) from /home/git/gitlab-shell/lib/gitlab_shell.rb:63:in `validate_access' from /home/git/gitlab-shell/lib/gitlab_shell.rb:24:in `exec' from /home/git/gitlab-shell/bin/gitlab-shell:16:in `<main>' fatal: Could not read from remote repository. Please make sure you have the correct access rights and the repository exists. ```
* | Revert "Revert "Merge branch 'git_hook_messages'""Valery Sizov2014-11-244-7/+33
| | | | | | | | This reverts commit f8453da5868dd7a23d0f2f3da7a45e33c441d1db.
* | Revert "Merge branch 'git_hook_messages'"Jacob Vosmaer2014-11-204-33/+7
| | | | | | | | | | | | | | | | | | | | At least the following things were broken: - missing require for 'gitlab_access_status' in lib/gitlab_net.rb - gitlabhq master internal API returns 'true' or 'false', gitlab-shell expects JSON This reverts commit 11311a95545f967a5736cd16ab5fc37f7e658519, reversing changes made to 45444597aef3e434571de2491934ae92357ad231.
* | Merge branch 'git_hook_messages'Dmitriy Zaporozhets2014-11-194-7/+33
|\ \ | | | | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com> Conflicts: VERSION
| * | Show error message when git access is rejectedValery Sizov2014-11-144-7/+33
| |/
* | Merge pull request #193 from cirosantilli/tag-branch-rm-double-dashDmitriy Zaporozhets2014-11-181-2/+2
|\ \ | |/ |/| Delete tags and branches that start with hyphen
| * Delete tags and branches that start with hyphenCiro Santilli2014-10-311-2/+2
| |
* | Fix the calls to pre- and post-receive custom hooksjkbzh2014-11-061-14/+41
| | | | | | | | | | | | - Reset G_ID if the custom pre-receive hook fails - Use a pipe to feed stdin to the custom pre- and post-receive hooks, in the same way that the standalone git works
* | Support for custom hooksDrew Blessing2014-11-053-3/+48
|/
* using true instead of 1jose2014-10-301-1/+1
|
* The gitlab pre-receive hook expects the call to exec to return 1jose2014-10-301-1/+1
| | | | | | | | | when a user has access. Doing an exit 0 effectively ended the thread. No code would be executed after the call to exec. This changes fixes the problem for the pre-receive hooks but you want to check that GitlabAccess.exec is not used elsewhere. A quick grep of the code seems to indicate this is not the case
* Merge pull request #81 from corny/patch-1Dmitriy Zaporozhets2014-10-281-3/+1
|\ | | | | avoid @user_tried
| * Update gitlab_shell.rbJulian K.2013-08-071-3/+1
| | | | | | using instance_variable_defined to avoid @user_tried
* | Merge pull request #177 from feedhenry/keys_listDmitriy Zaporozhets2014-10-281-0/+14
|\ \ | | | | | | Added list-keys command, with associated spec
| * | Added list-keys command and specDavid Martin2014-10-151-0/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Removed puts and tidied up regex Address the hound Address the hound, again Use single quotes Add back travis.yml file Remove travis.yml, only keep on fh-master Use single quotes Use single quotes
* | | Merge pull request #179 from feedhenry/projects_listDmitriy Zaporozhets2014-10-281-1/+9
|\ \ \ | | | | | | | | Added list-projects command & spec
| * | | Added list-projects command & specDavid Martin2014-10-151-1/+9
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | list-projects command usage Single quotes Use single quotes Use single quotes
* | | Merge branch 'master' of github.com:gitlabhq/gitlab-shellv2.1.0Dmitriy Zaporozhets2014-10-201-1/+2
|\ \ \ | |/ / | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com> Conflicts: .gitignore
| * | Split one instance variable per lineCiro Santilli2014-10-041-1/+2
| | |
* | | add secret tokenValery Sizov2014-10-151-1/+6
|/ /
* | Fix bug when changes to post-receive worker was not dumped properlyv2.0.1Dmitriy Zaporozhets2014-09-251-1/+1
| | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* | Add '.old' when renaming existing hooks directoryJacob Vosmaer2014-09-161-1/+1
| |
* | Symlink the whole hooks directoryJacob Vosmaer2014-09-161-4/+6
| |
* | Git remote remove only available in git > 1.7.12Dmitriy Zaporozhets2014-09-121-1/+1
| | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* | Make sure user got redis defaults if not setDmitriy Zaporozhets2014-09-121-0/+2
| | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* | support using custom redis database, fixes #173Tao2014-09-131-3/+4
| |
* | Merge pull request #91 from raphendyr/patch-1Dmitriy Zaporozhets2014-09-111-2/+6
|\ \ | | | | | | Replace build in home folder with environment
| * | Replace build in home folder with environmentJaakko Kantojärvi2013-09-131-2/+6
| | | | | | | | | | | | | | | You can't ever assume that user's home is /home/$USERNAME or /home/git Preferable is to assume that user running gitlab-shell has repositories in his home (get from the environment).
* | | Merge pull request #175 from yglukhov/security-issueDmitriy Zaporozhets2014-09-111-2/+20
|\ \ \ | | | | | | | | Security issue: imported URLs are stored along with password.
| * | | Security issue: imported URLs are stored along with password.uran2014-08-281-2/+20
| | | |
* | | | Merge pull request #141 from attiks/support-redis-passwordDmitriy Zaporozhets2014-09-111-1/+5
|\ \ \ \ | | | | | | | | | | added support for redis password
| * | | | added support for redis passwordPeter2014-03-211-1/+5
| | | | |
* | | | | GitLab /api/allowed endpoint requires POST requestDmitriy Zaporozhets2014-09-032-5/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit made changes to GitLab shell to work with huge pushed (ex. 1k branhes) using POST request to API Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* | | | | Update GitlabNet to match new logicDmitriy Zaporozhets2014-09-011-8/+4
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* | | | | Remove update hook logicDmitriy Zaporozhets2014-09-011-58/+0
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* | | | | GitlabAccess and GitlabPostReceive classes addedDmitriy Zaporozhets2014-09-013-3/+70
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Gitlab Access handles security check. GitlabPostReceive creates a sidekiq job Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* | | | | Use a different style of one-line classesJacob Vosmaer2014-08-271-1/+1
| | | | |
* | | | | Handle invalid number of argumentsJacob Vosmaer2014-08-271-3/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When a remote user with a valid SSH key runs something like 'ssh git@gitlab.example.com foobar', gitlab-shell would raise an exception in the GitlabShell#escape_path method. With this change, we catch an invalid number of arguments as soon as possible and exit.
* | | | | use abort isntead of raise when checking pathJob van der Voort2014-08-271-1/+1
| |/ / / |/| | |
* | | | Fix invalid repo_name/repo_path parsing by GitlabUpdateDmitriy Zaporozhets2014-08-132-12/+21
| | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* | | | Adding support+tests for annotated tags.Sean Edge2014-07-091-1/+6
| | | |
* | | | Merge pull request #155 from ssandler/masterDmitriy Zaporozhets2014-07-041-1/+1
|\ \ \ \ | | | | | | | | | | GitlabUpdate: use stripped path for repo_name
| * | | | GitlabUpdate: use stripped path for repo_nameScott Sandler2014-06-161-1/+1
| | | | |
* | | | | Improve coverage.Gabor Nagy2014-06-231-16/+19
|/ / / /
* | | | Expliclity require 'timeout' from the stdlibJacob Vosmaer2014-06-101-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | This avoids getting the following error on some platforms: NameError: uninitialized constant GitlabKeys::Timeout
* | | | Use authorized_keys lock when add new key to fileDmitriy Zaporozhets2014-05-311-3/+5
| | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>