Skip to content

feat(provider/gitlab): Add support for custom CAs#4941

Merged
Skarlso merged 4 commits intoexternal-secrets:mainfrom
girishsr25:gsr/feat-gitlab-ca-provider
Jun 27, 2025
Merged

feat(provider/gitlab): Add support for custom CAs#4941
Skarlso merged 4 commits intoexternal-secrets:mainfrom
girishsr25:gsr/feat-gitlab-ca-provider

Conversation

@girishsr25
Copy link
Copy Markdown
Contributor

@girishsr25 girishsr25 commented Jun 22, 2025

Problem Statement

The GitLab provider cannot connect to self-hosted instances that use custom TLS certificates, causing certificate validation errors.

Related Issue

Fixes #4938

Proposed Changes

This PR adds support for custom Certificate Authorities (CAs) to the GitLab provider by introducing caBundle and caProvider fields. This allows users to provide a trusted CA certificate, enabling secure connections to their self-hosted GitLab. The implementation aligns with existing patterns in other providers.

Checklist

  • I have read the contribution guidelines
  • All commits are signed with git commit --signoff
  • My changes have reasonable test coverage
  • All tests pass with make test
  • I ensured my PR is ready for review with make reviewable

Signed-off-by: girishsr25 <sgirish585@gmail.com>
@girishsr25 girishsr25 force-pushed the gsr/feat-gitlab-ca-provider branch from e20c81c to 47cb203 Compare June 22, 2025 08:52
@girishsr25 girishsr25 marked this pull request as ready for review June 22, 2025 08:54
@girishsr25 girishsr25 requested a review from a team as a code owner June 22, 2025 08:54
@girishsr25 girishsr25 requested a review from gusfcarvalho June 22, 2025 08:54
@girishsr25
Copy link
Copy Markdown
Contributor Author

@Skarlso Please do review when you get time, thanks

@Skarlso
Copy link
Copy Markdown
Contributor

Skarlso commented Jun 26, 2025

Perfect. Looks fantastic. :)

girishsr25 and others added 2 commits June 27, 2025 10:55
Signed-off-by: girishsr25 <sgirish585@gmail.com>
@sonarqubecloud
Copy link
Copy Markdown

@Skarlso Skarlso merged commit 70ec0c1 into external-secrets:main Jun 27, 2025
23 checks passed
alliseeisgold pushed a commit to alliseeisgold/external-secrets that referenced this pull request Jul 10, 2025
)

* lint: run make fmt and reviewable

Signed-off-by: girishsr25 <sgirish585@gmail.com>

* Update the docs

Signed-off-by: girishsr25 <sgirish585@gmail.com>

---------

Signed-off-by: girishsr25 <sgirish585@gmail.com>
Co-authored-by: Gergely Brautigam <182850+Skarlso@users.noreply.github.com>
Signed-off-by: asrormirzoev <asrormirzoev@yandex-team.ru>
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.

Support for CAProvider field in gitlab provider

2 participants