summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
* Merge branch 'readme-overview' into 'master'Douwe Maan2018-10-011-29/+19
|\ | | | | | | | | Update overview in readme See merge request gitlab-org/gitlab-shell!248
| * Update overview in readmeJacob Vosmaer2018-10-011-29/+19
|/
* Merge branch 'gitlab-shell-no-glid-argument' into 'master'Nick Thomas2018-10-012-22/+1
|\ | | | | | | | | | | | | Remove mistaken support for `gitlab-shell user-123` Closes #151 See merge request gitlab-org/gitlab-shell!247
| * Remove mistaken support for `gitlab-shell user-123`gitlab-shell-no-glid-argumentJacob Vosmaer2018-09-282-22/+1
| |
* | Merge branch '74-go-go-go-go-go' into 'master'Douwe Maan2018-09-287-19/+130
|\ \ | |/ |/| | | | | | | | | Feature flag for go/ruby gitlab-shell implementations Closes #74 See merge request gitlab-org/gitlab-shell!233
| * Allow the config directory to be specifiedNick Thomas2018-09-283-17/+31
| |
| * Specify a richer scheme to run the migration withNick Thomas2018-09-284-28/+60
| |
| * Run the Ruby specs in the "go:x.y" tests as wellNick Thomas2018-09-281-7/+25
| | | | | | | | | | | | Since the Go code now wraps the Ruby script, this makes sense and tests the interaction of the two codebases. Over time, the number of Ruby tests to run will approach 0.
| * Initial feature-flagged go/ruby switchNick Thomas2018-09-286-6/+53
|/
* Merge branch 'ash.mckenzie/display-feedback-v2' into 'master'Nick Thomas2018-09-259-88/+78
|\ | | | | | | | | Display helpful feedback when proxying an SSH git push to secondary request (v2) See merge request gitlab-org/gitlab-shell!246
| * Display helpful feedback when proxying an SSH git push to secondary request ↵Ash McKenzie2018-09-259-88/+78
|/ | | | (v2)
* Merge branch '155-merge-8-3-3-into-master' into 'master'Stan Hu2018-09-142-1/+4
|\ | | | | | | | | | | | | Merge 8-3-stable branch changes into master Closes #155 See merge request gitlab-org/gitlab-shell!245
| * Release v8.3.3v8.3.38-3-stableNick Thomas2018-09-142-1/+4
| |
* | Merge branch 'ash.mckenzie/display-feedback' into 'master'Stan Hu2018-09-135-25/+162
|\ \ | |/ |/| | | | | Display helpful feedback when proxying an SSH git push to secondary request See merge request gitlab-org/gitlab-shell!244
| * primary_repo can be a String or a Hashash.mckenzie/display-feedbackAsh McKenzie2018-09-131-34/+44
| |
| * Print message to $stderr for custom actionAsh McKenzie2018-09-133-0/+111
| |
| * Make ordered expectations around results outputAsh McKenzie2018-09-133-6/+21
| | | | | | | | This also cleans up the output from the tests as previously, Base64 encoded output was printed.
| * Consider 300 a successful response alsoAsh McKenzie2018-09-131-1/+2
| |
* | Merge branch '154-release-gitlab-shell-8-3-2' into 'master'Stan Hu2018-09-122-1/+5
|\ \ | | | | | | | | | | | | Release GitLab Shell v8.3.2 See merge request gitlab-org/gitlab-shell!243
| * | Release GitLab Shell v8.3.2sh-release-gitlab-shell-8-3-3Nick Thomas2018-09-122-1/+5
|/ /
* | Merge branch 'sh-fix-nonatomic-puts' into 'master'v8.3.2Nick Thomas2018-09-121-2/+3
|\ \ | |/ |/| | | | | | | | | Fix newlines not appearing between new log entries Closes gitlab-com/gl-infra/infrastructure#5017 See merge request gitlab-org/gitlab-shell!242
| * Fix newlines not appearing between new log entriessh-fix-nonatomic-putsStan Hu2018-09-111-2/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In https://gitlab.com/gitlab-com/gl-infra/infrastructure/issues/5017, we observed that lots of log messages were being dropped by Fluentd due to missing newlines. This occurs because there is a bug in Ruby where IO#puts calls write() twice: once to write the main text, and another to write the newline (https://bugs.ruby-lang.org/issues/14042). In a highly concurrent environment like GitLab.com, this can lead to interleaved newlines. A workaround is to use `IO#print` and append the newline ourselves. Closes https://gitlab.com/gitlab-com/gl-infra/infrastructure/issues/5017
* | Merge branch ↵Nick Thomas2018-09-1124-36/+662
|\ \ | |/ |/| | | | | | | | | | | | | '154-ensure-http-status-codes-are-integers-add-missing-specs-for-various-handled-http-status-code' into 'master' Ensure text/plain & text/html content typs are handled and add missing specs for handled HTTP status codes Closes #154 See merge request gitlab-org/gitlab-shell!239
| * Ensure text/plain & text/html content typs are handled and add missing specs ↵Ash McKenzie2018-09-1124-36/+662
|/ | | | for handled HTTP status codes
* Merge branch 'sh-release-8.3.1' into 'master'v8.3.1Stan Hu2018-09-102-1/+4
|\ | | | | | | | | Release 8.3.1 to fix version numbering See merge request gitlab-org/gitlab-shell!241
| * Release 8.3.1 to fix version numberingStan Hu2018-09-102-1/+4
|/
* Merge branch 'sh-release-8.3.0' into 'master'Stan Hu2018-09-072-1/+4
|\ | | | | | | | | Release v8.3.0 See merge request gitlab-org/gitlab-shell!240
| * Release v8.3.0sh-release-8.3.0Stan Hu2018-09-072-1/+4
|/ | | | Includes custom action support for proxying SSH to a Geo secondary
* Merge branch 'ash.mckenzie/custom-action-support' into 'master'v8.3.0Stan Hu2018-09-0716-56/+618
|\ | | | | | | | | Custom Action support See merge request gitlab-org/gitlab-shell!215
| * Only decode result body if it has content upon custom action failureStan Hu2018-09-071-1/+3
| |
| * Custom Action supportash.mckenzie/custom-action-supportAsh McKenzie2018-09-0811-17/+564
| |
| * GitlabAccessStatus needs HTTP response status codeAsh McKenzie2018-09-073-13/+20
| |
| * Use constants for git commands in GitlabShellAsh McKenzie2018-09-071-6/+11
| |
| * New HTTPCodes moduleAsh McKenzie2018-09-071-0/+3
| |
| * GitAccessStatus initializer sensible defaultsAsh McKenzie2018-09-072-9/+2
| |
| * Move some GitlabNet requires into HTTPHelperAsh McKenzie2018-09-072-2/+2
| |
| * GitlabConfig, GitlabShell & GitlabKeys spec fixesAsh McKenzie2018-09-073-11/+16
| | | | | | | | GitlabConfig spec was relying upon real config.yml
* | Merge branch 'sh-release-8.2.1' into 'master'v8.2.1Nick Thomas2018-09-072-1/+4
|\ \ | |/ |/| | | | | Release gitlab-shell v8.2.1 See merge request gitlab-org/gitlab-shell!238
| * Release gitlab-shell v8.2.1sh-release-8.2.1Stan Hu2018-09-062-1/+4
|/
* Merge branch 'ash.mckenzie/allowed-api-proper-http-status-code-support' into ↵Stan Hu2018-09-073-1/+11
|\ | | | | | | | | | | | | | | | | 'master' Proper HTTP status code support for /api/v4/allowed API endpoint Closes gitlab-ce#51211 See merge request gitlab-org/gitlab-shell!237
| * /api/v4/allowed returns proper HTTP status codesAsh McKenzie2018-09-071-1/+2
| | | | | | | | | | * Previously, a 200 (OK) was sent when the user was unauthorised or the project was not found (or the user didn't have access) * We still treat 401 and 404 as 'success' but we need to explicitly handle them
| * New HTTPCodes moduleAsh McKenzie2018-09-072-0/+9
|/
* Merge branch '153-release-8-2-0' into 'master'v8.2.0Robert Speicher2018-09-052-1/+5
|\ | | | | | | | | | | | | Release gitlab-shell v8.2.0 Closes #153 See merge request gitlab-org/gitlab-shell!236
| * Release gitlab-shell v8.2.0Nick Thomas2018-09-052-1/+5
|/
* Merge branch 'zj-fix-non-200-resp-code' into 'master'Nick Thomas2018-09-053-2/+10
|\ | | | | | | | | | | | | Fix handling non 200 response codes Closes #152 See merge request gitlab-org/gitlab-shell!235
| * Fix handling non 200 response codeszj-fix-non-200-resp-codeZeger-Jan van de Weg2018-09-053-2/+10
|/ | | | | | | | | After the cleanup in https://gitlab.com/gitlab-org/gitlab-shell/merge_requests/231 gitlab-shell mishandled the non 200 response codes. This commit removes another few lines of codes, which fixes this. Also now we test against this case through mocking. Fixes https://gitlab.com/gitlab-org/gitlab-shell/issues/152
* Merge branch 'zj-cleanup-exec' into 'master'Nick Thomas2018-08-244-223/+52
|\ | | | | | | | | | | | | Cleanup `git-upload-*` and `git-receive-*` related code Closes gitaly#1300 See merge request gitlab-org/gitlab-shell!232
| * Clean up cmd_exec execution environmentZeger-Jan van de Weg2018-08-244-198/+43
| | | | | | | | | | | | | | | | | | | | | | Given the gitaly-* now proxy the data from the client to the Gitaly server, the environment variables aren't used. Therefor we don't have to set them either. Only exception to the rule, is the GITALY_TOKEN. These changes also remove the `GIT_TRACE` options, introduced by 192e2bd367494bf66746c8971896a2d9cb84fc92. Part of: https://gitlab.com/gitlab-org/gitaly/issues/1300
| * Remove non Gitaly code pathsZeger-Jan van de Weg2018-08-202-39/+23
|/ | | | | | | All shell access goes through Gitaly, so dead code paths exist to support the legacy way too. This change mostly removes the dead code from `#process_cmd`.
* Merge branch 'zj-remove-repo-path' into 'master'Nick Thomas2018-08-174-43/+7
|\ | | | | | | | | | | | | Remove repo_path from GitlabShell Closes #135 See merge request gitlab-org/gitlab-shell!231