summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Add context fields to loggingid-context-fieldsIgor Drozdov2021-09-156-13/+13
| | | | It adds correlation ids wherever possible
* Merge branch 'id-sshd-test-3' into 'main'Nick Thomas2021-09-154-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.goIgor Drozdov2021-09-154-73/+217
|/
* Merge branch 'id-session-test-2' into 'main'Nick Thomas2021-09-151-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-2Igor Drozdov2021-09-151-0/+145
|/
* Merge branch 'id-session-test' into 'main'Nick Thomas2021-09-151-0/+44
|\ | | | | | | | | Unit test sshd.handleEnv function See merge request gitlab-org/gitlab-shell!521
| * Unit test sshd.handleEnv functionIgor Drozdov2021-09-151-0/+44
|/
* Merge branch 'id-sshd-tests' into 'main'Nick Thomas2021-09-151-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-testsIgor Drozdov2021-09-151-0/+16
|/
* Merge branch 'refactor/cmd' into 'main'Nick Thomas2021-09-1419-489/+737
|\ | | | | | | | | | | | | refactor: rearchitect command and executable Go modules Closes #214 See merge request gitlab-org/gitlab-shell!514
| * Merge branch 'main' into refactor/cmdNick Thomas2021-09-144-3/+6
| |\ | |/ |/|
* | Merge branch 'fix-bad-spec' into 'main'Igor Drozdov2021-09-141-1/+1
|\ \ | | | | | | | | | | | | Fix a flaky test See merge request gitlab-org/gitlab-shell!519
| * | Fix a flaky testNick Thomas2021-09-141-1/+1
|/ /
* | Merge branch 'eread/update-project-tool-versions' into 'main'Ash McKenzie2021-09-142-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 toolingEvan Read2021-09-142-1/+3
|/ /
* | Merge branch 'build/make' into 'main'Igor Drozdov2021-09-091-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 Makefilefeistel2021-09-071-1/+2
| | |
| | * refactor: fix style issuesfeistel2021-09-082-5/+4
| | |
| | * refactor: cleanup func signature and remove unused argsfeistel2021-09-089-29/+22
| | |
| | * refactor: rearchitect command and executable Go modulesfeistel2021-09-0819-483/+739
| |/ |/|
* | Merge branch 'remove/generic-args' into 'main'Nick Thomas2021-09-0811-39/+41
|\ \ | | | | | | | | | | | | | | | | | | refactor: remove commandargs.GenericArgs Closes #212 See merge request gitlab-org/gitlab-shell!506
| * | refactor: add acceptargs field to executablefeistel2021-09-089-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 messagefeistel2021-09-081-1/+2
| | |
| * | refactor: remove commandargs.GenericArgsfeistel2021-08-113-21/+18
| | |
* | | Merge branch 'remove/ioutil' into 'main'Igor Drozdov2021-09-0824-73/+68
|\ \ \ | |_|/ |/| | | | | | | | refactor: move away from ioutil (deprecated) See merge request gitlab-org/gitlab-shell!511
| * | refactor: move away from ioutil (deprecated)feistel2021-08-1924-73/+68
| | |
* | | Merge branch 'release-13-21' into 'main'v13.21.0Nick Thomas2021-08-252-1/+19
|\ \ \ | |/ / |/| | | | | | | | Release v13.21.0 See merge request gitlab-org/gitlab-shell!512
| * | Version 3.21.0Nick Thomas2021-08-252-1/+11
| | |
| * | Add missing CHANGELOG entriesNick Thomas2021-08-251-0/+8
|/ /
* | Merge branch 'bump/go-1.16' into 'main'Nick Thomas2021-08-193-17/+1
|\ \ | | | | | | | | | | | | build: bump go version to 1.16 See merge request gitlab-org/gitlab-shell!509
| * | ci: remove go 1.15 test jobfeistel2021-08-191-7/+0
| | |
| * | build: bump go version to 1.16feistel2021-08-192-10/+1
| | |
* | | Merge branch 'test/go-1.17' into 'main'Patrick Bajao2021-08-191-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 pipelinesfeistel2021-08-191-1/+8
|/ /
* | Merge branch 'verify/cafile' into 'main'Ash McKenzie2021-08-182-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 creationAsh McKenzie2021-08-171-7/+7
| | |
| * | fix: make sure ErrCafileNotFound is returned only when the file doesn't existfeistel2021-08-111-1/+4
| | |
| * | fix: validate client cert paths exist on disk before proceedingfeistel2021-08-112-29/+44
| |/
* | Merge branch 'update/newclientopts' into 'main'Nick Thomas2021-08-135-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 errorfeistel2021-08-133-7/+12
| | |
| * | refactor: update usage of NewHTTPClient to NewHTTPClientWithOptsfeistel2021-08-113-4/+12
| |/
* | Merge branch 'bump/go1.15' into 'main'Patrick Bajao2021-08-123-9/+1
|\ \ | |/ |/| | | | | build: bump go to 1.15 See merge request gitlab-org/gitlab-shell!505
| * test: remove go 1.14 test jobfeistel2021-08-111-7/+0
| |
| * build: bump go to 1.15feistel2021-08-112-2/+1
|/
* Merge branch '499-use-labkit-logging-initialize' into 'main'Patrick Bajao2021-08-0410-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 setupNick Thomas2021-08-0410-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 Bajao2021-08-042-4/+9
|\ \ | |/ |/| | | | | Modify regex to prevent partial matches See merge request gitlab-org/security/gitlab-shell!6
| * Modify regex to prevent partial matchesRobert May2021-06-292-4/+9
| |
* | Merge branch '477-remove-logrus-tests' into 'main'Igor Drozdov2021-08-025-136/+77
|\ \ | | | | | | | | | | | | | | | | | | Remove some unreliable tests Closes #477 See merge request gitlab-org/gitlab-shell!503
| * | Remove some unreliable testsNick Thomas2021-07-305-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.