Skip to content

[v11] Add mutex for certs in local proxy #18623

Merged
GavinFrazar merged 2 commits intobranch/v11from
gavinfrazar/v11-fix-data-race-in-local-proxy
Nov 22, 2022
Merged

[v11] Add mutex for certs in local proxy #18623
GavinFrazar merged 2 commits intobranch/v11from
gavinfrazar/v11-fix-data-race-in-local-proxy

Conversation

@GavinFrazar
Copy link
Copy Markdown
Contributor

Backports #18278 to v11

* Add mutex for certs in local proxy

* Update lib/srv/alpnproxy/local_proxy.go

Co-authored-by: Ryan Clark <ryan.clark@goteleport.com>

* Update lib/srv/alpnproxy/local_proxy.go

Co-authored-by: Ryan Clark <ryan.clark@goteleport.com>

* Update lib/srv/alpnproxy/local_proxy.go

Co-authored-by: Ryan Clark <ryan.clark@goteleport.com>

* Move cert checking out of middleware into local proxy

* Configure a logger for local proxy

* Fixup imports

* Add tests for local proxy

* test for data race
* test for cert checking

* Update integration test for local proxy

* Mark err assert fns as helpers

Co-authored-by: Ryan Clark <ryan.clark@goteleport.com>
@github-actions github-actions Bot removed the request for review from rosstimothy November 21, 2022 11:21
@GavinFrazar GavinFrazar enabled auto-merge (squash) November 22, 2022 07:49
@GavinFrazar GavinFrazar merged commit 4b43183 into branch/v11 Nov 22, 2022
@GavinFrazar GavinFrazar deleted the gavinfrazar/v11-fix-data-race-in-local-proxy branch November 22, 2022 08:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants