Skip to content
New issue

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

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

Already on GitHub? # to your account

[fix] Correctly initialize detectors with cloud endpoint customization #3333

Merged
merged 3 commits into from
Sep 25, 2024

Conversation

mcastorina
Copy link
Collaborator

Description:

We were only initializing if the detector was configured with a custom endpoint, but not in the default case.

Checklist:

  • Tests passing (make test-community)?
  • Lint passing (make lint this requires golangci-lint)?

We were only initializing if the detector was configured with a custom
endpoint, but not in the default case.
@mcastorina mcastorina requested a review from a team as a code owner September 25, 2024 17:24
@mcastorina mcastorina requested a review from a team as a code owner September 25, 2024 18:31
Copy link
Collaborator

@trufflesteeeve trufflesteeeve left a comment

Choose a reason for hiding this comment

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

LGTM

@mcastorina mcastorina merged commit 4484bf4 into main Sep 25, 2024
12 checks passed
@mcastorina mcastorina deleted the fix-endpoint-customizer-init branch September 25, 2024 20:06
@kashifkhan0771
Copy link
Contributor

Thanks @mcastorina!

kashifkhan0771 pushed a commit to kashifkhan0771/trufflehog that referenced this pull request Oct 1, 2024
trufflesecurity#3333)

* [fix] Correctly initialize detectors with cloud endpoint customization

We were only initializing if the detector was configured with a custom
endpoint, but not in the default case.

* Add test

* Fix gitlab.v2 detector
@rgmz rgmz mentioned this pull request Nov 12, 2024
2 tasks
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants