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

Add support for mutual authentication via TLS #459

Merged
merged 1 commit into from
Aug 12, 2024

Conversation

shearn89
Copy link
Contributor

@shearn89 shearn89 commented Aug 9, 2024

Prior to this change, there was no way to pass in a client certificate if you had set up Nexus behind a load balancer with mutual auth configured.

This change exposes some new configuration values that allow you to pass in a path to a file on disk containing your client key/cert (which can be in a single file) and optionally a Root CA.

Follows on from datadrivers/go-nexus-client#146

@shearn89
Copy link
Contributor Author

shearn89 commented Aug 9, 2024

make docs is failing on unrelated code for me:

internal/services/repository/resource_repository_docker_hosted.go:48:12: cannot use repository.HostedStorage{…} (value of type "github.com/datadrivers/go-nexus-client/nexus3/schema/repository".HostedStorage) as "github.com/datadrivers/go-nexus-client/nexus3/schema/repository".DockerHostedStorage value in struct literal
internal/services/repository/resource_repository_docker_hosted.go:112:61: cannot use &repo.Storage (value of type *"github.com/datadrivers/go-nexus-client/nexus3/schema/repository".DockerHostedStorage) as *"github.com/datadrivers/go-nexus-client/nexus3/schema/repository".HostedStorage value in argument to flattenHostedStorage

@shearn89 shearn89 marked this pull request as ready for review August 9, 2024 14:57
@anmoel
Copy link
Member

anmoel commented Aug 9, 2024

@shearn89 it is related to my change. i will merge your PR after the implementation of the issue #341.

@anmoel
Copy link
Member

anmoel commented Aug 12, 2024

@shearn89 please rebase your PR

Prior to this change, there was no way to pass in a client certificate
if you had set up Nexus behind a load balancer with mutual auth
configured.

This change exposes some new configuration values that allow you to pass
in a path to a file on disk containing your client key/cert (which can
be in a single file) and optionally a Root CA.
Copy link
Member

@anmoel anmoel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm
lgtm

@anmoel anmoel merged commit 24a1eb2 into datadrivers:main Aug 12, 2024
3 checks passed
@shearn89 shearn89 deleted the feat/mutual-tls branch August 12, 2024 12:44
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.

2 participants