Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Create mTLS functionality for promxy #445

Merged
merged 8 commits into from
Jul 2, 2021

Conversation

jerrybelmonte
Copy link
Contributor

Promxy was not leveraging mTLS. We tried to mimic the Prometheus approach using a web.config.file command line argument.

This resolves issue #432

Let us know if there's anything we need to fix, or if we can help speed up the acceptance process in any way.

jerrybelmonte and others added 6 commits June 22, 2021 16:11
Signed-off-by: Kira Boyle <kboyle@pivotal.io>
Signed-off-by: Devon Warshaw <warshawd@vmware.com>
Signed-off-by: Kira Boyle <kboyle@pivotal.io>
Signed-off-by: Devon Warshaw <warshawd@vmware.com>
…aceholder command

Signed-off-by: Devon Warshaw <warshawd@vmware.com>
Signed-off-by: Kira Boyle <kboyle@pivotal.io>
Signed-off-by: Robert Sullivan <rsullivan@pivotal.io>
@jacksontj
Copy link
Owner

The PR seems okay -- there looks to be some minor failures in ci (likely just a make fmt and make imports required).

@jerrybelmonte
Copy link
Contributor Author

Thanks for the tip on the requirements! Went ahead and applied the requirements to hopefully fix the minor failures.

@jacksontj jacksontj merged commit 0fcb1a1 into jacksontj:master Jul 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants