Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Add context fields to loggingid-context-fields | Igor Drozdov | 2021-09-15 | 6 | -13/+13 |
| | | | | It adds correlation ids wherever possible | ||||
* | Merge branch 'id-sshd-test-3' into 'main' | Nick Thomas | 2021-09-15 | 4 | -73/+217 |
|\ | | | | | | | | | | | | | Extract server config related code out of sshd.go Closes #523 See merge request gitlab-org/gitlab-shell!523 | ||||
| * | Extract server config related code out of sshd.go | Igor Drozdov | 2021-09-15 | 4 | -73/+217 |
|/ | |||||
* | Merge branch 'id-session-test-2' into 'main' | Nick Thomas | 2021-09-15 | 1 | -0/+145 |
|\ | | | | | | | | | | | | | Unit test exit-codes for sshd/session.go Closes #522 See merge request gitlab-org/gitlab-shell!522 | ||||
| * | Unit test exit-codes for sshd/session.goid-session-test-2 | Igor Drozdov | 2021-09-15 | 1 | -0/+145 |
|/ | |||||
* | Merge branch 'id-session-test' into 'main' | Nick Thomas | 2021-09-15 | 1 | -0/+44 |
|\ | | | | | | | | | Unit test sshd.handleEnv function See merge request gitlab-org/gitlab-shell!521 | ||||
| * | Unit test sshd.handleEnv function | Igor Drozdov | 2021-09-15 | 1 | -0/+44 |
|/ | |||||
* | Merge branch 'id-sshd-tests' into 'main' | Nick Thomas | 2021-09-15 | 1 | -0/+16 |
|\ | | | | | | | | | Add TestInvalidClientConfig and TestNewServerWithoutHosts for sshd.go See merge request gitlab-org/gitlab-shell!518 | ||||
| * | Add TestInvalidClientConfig and TestNewServerWithoutHosts for sshd.goid-sshd-tests | Igor Drozdov | 2021-09-15 | 1 | -0/+16 |
|/ | |||||
* | Merge branch 'refactor/cmd' into 'main' | Nick Thomas | 2021-09-14 | 19 | -489/+737 |
|\ | | | | | | | | | | | | | refactor: rearchitect command and executable Go modules Closes #214 See merge request gitlab-org/gitlab-shell!514 | ||||
| * | Merge branch 'main' into refactor/cmd | Nick Thomas | 2021-09-14 | 4 | -3/+6 |
| |\ | |/ |/| | |||||
* | | Merge branch 'fix-bad-spec' into 'main' | Igor Drozdov | 2021-09-14 | 1 | -1/+1 |
|\ \ | | | | | | | | | | | | | Fix a flaky test See merge request gitlab-org/gitlab-shell!519 | ||||
| * | | Fix a flaky test | Nick Thomas | 2021-09-14 | 1 | -1/+1 |
|/ / | |||||
* | | Merge branch 'eread/update-project-tool-versions' into 'main' | Ash McKenzie | 2021-09-14 | 2 | -1/+3 |
|\ \ | | | | | | | | | | | | | Update Ruby version to 2.7.4 and add Go version 1.16.8 for tooling See merge request gitlab-org/gitlab-shell!517 | ||||
| * | | Update Ruby version to 2.7.4 and add Go version 1.16.8 for tooling | Evan Read | 2021-09-14 | 2 | -1/+3 |
|/ / | |||||
* | | Merge branch 'build/make' into 'main' | Igor Drozdov | 2021-09-09 | 1 | -1/+2 |
|\ \ | | | | | | | | | | | | | build: move build task to the top of the Makefile See merge request gitlab-org/gitlab-shell!513 | ||||
| * | | build: move build task to the top of the Makefile | feistel | 2021-09-07 | 1 | -1/+2 |
| | | | |||||
| | * | refactor: fix style issues | feistel | 2021-09-08 | 2 | -5/+4 |
| | | | |||||
| | * | refactor: cleanup func signature and remove unused args | feistel | 2021-09-08 | 9 | -29/+22 |
| | | | |||||
| | * | refactor: rearchitect command and executable Go modules | feistel | 2021-09-08 | 19 | -483/+739 |
| |/ |/| | |||||
* | | Merge branch 'remove/generic-args' into 'main' | Nick Thomas | 2021-09-08 | 11 | -39/+41 |
|\ \ | | | | | | | | | | | | | | | | | | | refactor: remove commandargs.GenericArgs Closes #212 See merge request gitlab-org/gitlab-shell!506 | ||||
| * | | refactor: add acceptargs field to executable | feistel | 2021-09-08 | 9 | -20/+24 |
| | | | | | | | | | | | | | | | parse logic will only run if the executable accept args. healthcheck is the only one not accepting arguments. | ||||
| * | | refactor: improve unknown executable error message | feistel | 2021-09-08 | 1 | -1/+2 |
| | | | |||||
| * | | refactor: remove commandargs.GenericArgs | feistel | 2021-08-11 | 3 | -21/+18 |
| | | | |||||
* | | | Merge branch 'remove/ioutil' into 'main' | Igor Drozdov | 2021-09-08 | 24 | -73/+68 |
|\ \ \ | |_|/ |/| | | | | | | | | refactor: move away from ioutil (deprecated) See merge request gitlab-org/gitlab-shell!511 | ||||
| * | | refactor: move away from ioutil (deprecated) | feistel | 2021-08-19 | 24 | -73/+68 |
| | | | |||||
* | | | Merge branch 'release-13-21' into 'main'v13.21.0 | Nick Thomas | 2021-08-25 | 2 | -1/+19 |
|\ \ \ | |/ / |/| | | | | | | | | Release v13.21.0 See merge request gitlab-org/gitlab-shell!512 | ||||
| * | | Version 3.21.0 | Nick Thomas | 2021-08-25 | 2 | -1/+11 |
| | | | |||||
| * | | Add missing CHANGELOG entries | Nick Thomas | 2021-08-25 | 1 | -0/+8 |
|/ / | |||||
* | | Merge branch 'bump/go-1.16' into 'main' | Nick Thomas | 2021-08-19 | 3 | -17/+1 |
|\ \ | | | | | | | | | | | | | build: bump go version to 1.16 See merge request gitlab-org/gitlab-shell!509 | ||||
| * | | ci: remove go 1.15 test job | feistel | 2021-08-19 | 1 | -7/+0 |
| | | | |||||
| * | | build: bump go version to 1.16 | feistel | 2021-08-19 | 2 | -10/+1 |
| | | | |||||
* | | | Merge branch 'test/go-1.17' into 'main' | Patrick Bajao | 2021-08-19 | 1 | -1/+8 |
|\ \ \ | |/ / |/| | | | | | | | | ci: start integrating go 1.17 into the CI pipelines See merge request gitlab-org/gitlab-shell!510 | ||||
| * | | ci: start integrating go 1.17 into the CI pipelines | feistel | 2021-08-19 | 1 | -1/+8 |
|/ / | |||||
* | | Merge branch 'verify/cafile' into 'main' | Ash McKenzie | 2021-08-18 | 2 | -29/+47 |
|\ \ | | | | | | | | | | | | | | | | | | | fix: validate client cert paths exist on disk before proceeding Closes #486 See merge request gitlab-org/gitlab-shell!508 | ||||
| * | | test: move os.stat check before the hcc creation | Ash McKenzie | 2021-08-17 | 1 | -7/+7 |
| | | | |||||
| * | | fix: make sure ErrCafileNotFound is returned only when the file doesn't exist | feistel | 2021-08-11 | 1 | -1/+4 |
| | | | |||||
| * | | fix: validate client cert paths exist on disk before proceeding | feistel | 2021-08-11 | 2 | -29/+44 |
| |/ | |||||
* | | Merge branch 'update/newclientopts' into 'main' | Nick Thomas | 2021-08-13 | 5 | -9/+22 |
|\ \ | | | | | | | | | | | | | | | | | | | refactor: update usage of NewHTTPClient to NewHTTPClientWithOpts Closes #484 See merge request gitlab-org/gitlab-shell!507 | ||||
| * | | refactor: change httpclient to return an error | feistel | 2021-08-13 | 3 | -7/+12 |
| | | | |||||
| * | | refactor: update usage of NewHTTPClient to NewHTTPClientWithOpts | feistel | 2021-08-11 | 3 | -4/+12 |
| |/ | |||||
* | | Merge branch 'bump/go1.15' into 'main' | Patrick Bajao | 2021-08-12 | 3 | -9/+1 |
|\ \ | |/ |/| | | | | | build: bump go to 1.15 See merge request gitlab-org/gitlab-shell!505 | ||||
| * | test: remove go 1.14 test job | feistel | 2021-08-11 | 1 | -7/+0 |
| | | |||||
| * | build: bump go to 1.15 | feistel | 2021-08-11 | 2 | -2/+1 |
|/ | |||||
* | Merge branch '499-use-labkit-logging-initialize' into 'main' | Patrick Bajao | 2021-08-04 | 10 | -43/+98 |
|\ | | | | | | | | | | | | | Switch to labkit for logging system setup Closes #271 See merge request gitlab-org/gitlab-shell!504 | ||||
| * | Switch to labkit for logging system setup | Nick Thomas | 2021-08-04 | 10 | -43/+98 |
| | | | | | | | | | | | | | | | | | | | | | | - We start supporting the "color" format for logs. - We now respond to SIGHUP by reopening the log file. - We now respect the log format when no log filename is specified. Output to syslog in the event of logging system setup is preserved in OpenSSH mode. Changelog: added | ||||
* | | Merge branch 'security-300265' into 'main' | Patrick Bajao | 2021-08-04 | 2 | -4/+9 |
|\ \ | |/ |/| | | | | | Modify regex to prevent partial matches See merge request gitlab-org/security/gitlab-shell!6 | ||||
| * | Modify regex to prevent partial matches | Robert May | 2021-06-29 | 2 | -4/+9 |
| | | |||||
* | | Merge branch '477-remove-logrus-tests' into 'main' | Igor Drozdov | 2021-08-02 | 5 | -136/+77 |
|\ \ | | | | | | | | | | | | | | | | | | | Remove some unreliable tests Closes #477 See merge request gitlab-org/gitlab-shell!503 | ||||
| * | | Remove some unreliable tests | Nick Thomas | 2021-07-30 | 5 | -136/+77 |
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | Logrus buffers its output internally, which makes these tests fail intermittently. They're also not a good example to follow generally. We now have acceptance tests that exercise this functionality so I'm pretty relaxed about losing the expectations. However, we can test them by inspecting the server-received metadata too, so there's no loss of coverage here. The move from logrus to labkit for logging also makes these tests hard to justify keeping. |