Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Generate and log correlation IDs | Stan Hu | 2020-07-31 | 6 | -20/+36 |
| | | | | | This will make it easier to tie an SSH access request to Rails API and Gitaly requests. | ||||
* | Revert "Update executable.go" | Igor Drozdov | 2020-07-23 | 1 | -0/+22 |
| | | | This reverts commit 869aeb9057962b089abfd8ce0b6d4a0962bbb154 | ||||
* | Update executable.go | Igor Drozdov | 2020-07-23 | 1 | -22/+0 |
| | |||||
* | Log SSH key details | Stan Hu | 2020-07-23 | 6 | -2/+14 |
| | | | | | | | | | | | | | | | | | | | | | Right now when a client such as gitlab-shell calls the `/api/v4/internal/allowed` API, the response only tells the client what user has been granted access, and it's impossible to tell which deploy key/token was used in the authentication request. This commit adds logs for the following when available: 1. `gl_key_type` (e.g. `deploy_key` or `key`) 2. `gl_key_id` These fields make it possible for admins to identify the exact record that was used to authenticate the user. API changes in the `/internal/allowed` endpoint in https://gitlab.com/gitlab-org/gitlab/-/merge_requests/37289 are needed to support this. Relates to https://gitlab.com/gitlab-org/gitlab-shell/-/issues/203 | ||||
* | Log remote IP for executed commandssh-log-remote-ip | Stan Hu | 2020-07-20 | 2 | -1/+8 |
| | | | | | | | | Admins may want to know what client IP originated the request. This commit adds a `remote_ip` field to the log that extracts the IP address from the `SSH_CONNECTION` environment variable. Closes https://gitlab.com/gitlab-org/gitlab-shell/-/issues/199 | ||||
* | Pass in ssl_cert_dir config setting | Ash McKenzie | 2020-07-02 | 4 | -4/+36 |
| | |||||
* | Include SSL_CERT_DIR env var in command | Ash McKenzie | 2020-07-02 | 2 | -22/+60 |
| | |||||
* | Support new ssl_cert_dir config setting | Ash McKenzie | 2020-07-01 | 2 | -0/+10 |
| | |||||
* | Set client name when making requests to Gitalycl-client-name | Changzheng Liu | 2020-05-21 | 1 | -3/+16 |
| | |||||
* | Fix race conditions with logrus testingsh-fix-logrus-race | Stan Hu | 2020-05-11 | 3 | -0/+19 |
| | | | | | | | | | logrus fires a Goroutine to write logs, so the tests could fail if they checked the event queue before the logrus have fired. Since there isn't an easy way to flush all outstanding hooks, we just retry every 100 ms for up to a second for log to arrive in the queue. Closes https://gitlab.com/gitlab-org/gitlab-shell/-/issues/450 | ||||
* | Fix race conditions in testssh-add-http-status-code | Stan Hu | 2020-05-08 | 3 | -14/+15 |
| | | | | | | | | Calling logrus hook.LastEntry() can lead to race conditions. Use AllEntries instead: https://github.com/sirupsen/logrus/blob/60c74ad9be0d874af0ab0daef6ab07c5c5911f0d/hooks/test/test.go#L77 Closes https://gitlab.com/gitlab-org/gitlab-shell/-/issues/450 | ||||
* | Ensure we are passing the parsed secret | DJ Mountney | 2020-05-06 | 1 | -1/+1 |
| | | | | | | | | Rather than the secret file. The parsing of the file was already done in the gitlab-shell config. This fixes an issue where a recent refactor of the gitlabnet client passed the wrong value. | ||||
* | Move gitlabnet client to client packagejc-refactor-gitlabnet-client | John Cai | 2020-05-04 | 36 | -938/+69 |
| | |||||
* | Geo Pull custom action support | Ash McKenzie | 2020-04-17 | 6 | -10/+201 |
| | |||||
* | Allow allowedPayloadPath to be provided | Ash McKenzie | 2020-04-17 | 1 | -7/+11 |
| | |||||
* | Rename Geo Push custom action test | Ash McKenzie | 2020-04-17 | 1 | -1/+1 |
| | |||||
* | Rename action to be more accurate | Ash McKenzie | 2020-04-17 | 1 | -5/+5 |
| | |||||
* | Rename allowed payload JSON for accuracy | Ash McKenzie | 2020-04-17 | 2 | -2/+2 |
| | |||||
* | New pktline package | Ash McKenzie | 2020-04-17 | 2 | -0/+162 |
| | | | | | | | Package is responsible for parsing git pkt lines. Copied from gitaly, for now. | ||||
* | Extract customaction into a separate moduleid-extract-custom-action-in-separate-module | Igor Drozdov | 2020-04-14 | 5 | -45/+90 |
| | | | | We'll reuse this module for uploadpack in the future | ||||
* | Add missed protocol againsh-log-git-upload-receive-pack | David Kim | 2020-04-08 | 1 | -1/+1 |
| | |||||
* | Remove unnecessary command argument | David Kim | 2020-04-08 | 4 | -5/+5 |
| | |||||
* | Move logging to handler instead | David Kim | 2020-04-07 | 5 | -29/+23 |
| | |||||
* | Change git command logging keys to be be snake cased | David Kim | 2020-04-03 | 1 | -5/+5 |
| | |||||
* | Add test for command logging | David Kim | 2020-04-03 | 7 | -38/+35 |
| | |||||
* | Add tests for logging | David Kim | 2020-03-30 | 3 | -0/+24 |
| | |||||
* | Add git-archive-pack | Stan Hu | 2020-03-26 | 3 | -12/+23 |
| | |||||
* | Log git-{upload-pack,receive-pack} requests | Stan Hu | 2020-03-26 | 2 | -0/+24 |
| | | | | This restores the previous Ruby gitlab-shell behavior. | ||||
* | Log internal HTTP requests | Stan Hu | 2020-03-10 | 5 | -18/+168 |
| | | | | | | | This restores the previous behavior of logging the success and failures of internal HTTP requests. Part of https://gitlab.com/gitlab-org/gitlab/issues/207916 | ||||
* | commands: pass through GIT_PROTOCOL envvar provided by clientsps-git-protocol-envvar | Patrick Steinhardt | 2020-02-28 | 3 | -2/+6 |
| | | | | | | | | | | | Both git-upload-pack and git-receive-pack services inspect the GIT_PROTOCOL environment transferred via SSH in order to decide which protocols are supported by a given client. Currently, we don't use the environment variable at all, though, but instead forward the GitProtocol field of the access verification response. Improve this by passing on the GIT_PROTOCOL environment variable provided by the client as-is. | ||||
* | Remove support for Custom data.info_message210-remove-action-custom-inform_client-once-12-3-has-been-released | Ash McKenzie | 2019-12-24 | 5 | -10/+1 |
| | |||||
* | Fix Typos | flowed | 2019-12-21 | 1 | -1/+1 |
| | |||||
* | Add git-lfs upload operation assertion37371-git-clone-on-secondary-geo-node-fetches-lfs-files-from-primary | Ash McKenzie | 2019-12-03 | 1 | -1/+3 |
| | |||||
* | Use correct LFS download or upload operation names | Ash McKenzie | 2019-12-03 | 3 | -40/+62 |
| | |||||
* | Merge branch 'pass-ff-to-gitaly' into 'master' | Ash McKenzie | 2019-10-29 | 11 | -13/+101 |
| | | | | | Add support for Gitaly feature flags See merge request gitlab-org/gitlab-shell!351 | ||||
* | More consistent console messages (golang) | Ash McKenzie | 2019-10-23 | 6 | -19/+10 |
| | |||||
* | New console package for writing to the console | Ash McKenzie | 2019-10-23 | 2 | -0/+274 |
| | |||||
* | Rename import paths | Nick Thomas | 2019-10-18 | 56 | -209/+209 |
| | |||||
* | Move go code up one level | Nick Thomas | 2019-10-18 | 81 | -0/+5705 |