Skip to content

[Feat] Detector implementation for Azure Configuration Connection String Key #3939

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

Conversation

abmussani
Copy link
Contributor

Description:

This PR implements a new Detector for Azure Configuration Service. This detector looks for connection strings and verify it using Azure App Configuration REST API.

Test Output:

$ go test -tags=detectors ./pkg/detectors/azureappconfigconnectionstring/
ok  	github.com/trufflesecurity/trufflehog/v3/pkg/detectors/azureappconfigconnectionstring	4.684s

Checklist:

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

@abmussani abmussani requested review from a team as code owners February 26, 2025 16:54
@abmussani abmussani changed the title Azureappconfigconnectionstring detector impl [Feature] Detector implementation for Azure Configuration Connection String Key Feb 27, 2025
@abmussani abmussani changed the title [Feature] Detector implementation for Azure Configuration Connection String Key [Feat] Detector implementation for Azure Configuration Connection String Key Mar 5, 2025
@abmussani abmussani force-pushed the azureappconfigconnectionstring-detector-impl branch from ea5e0a8 to cb2fdaf Compare April 9, 2025 10:53
@abmussani abmussani force-pushed the azureappconfigconnectionstring-detector-impl branch from d0a4ac5 to f1121f3 Compare April 21, 2025 11:22
@zricethezav
Copy link
Collaborator

@abmussani just gotta fix merge conflicts

abmussani added 3 commits May 8, 2025 15:41
* main: (40 commits)
  [Feat] Detector implementation for Azure API Management Direct Management Key (trufflesecurity#3938)
  Fastly Analyzer (trufflesecurity#4082)
  Postman Code Uses Consistent Casing for Id Var Names (trufflesecurity#4124)
  Normalize UID to Uid in Postman Code (trufflesecurity#4125)
  postman_client.IDNameUUID becomes IdNameUid (trufflesecurity#4123)
  Fixed Kontent Detector (trufflesecurity#4122)
  [Fix] Added Prefix In Dockerhub Detector Regex (trufflesecurity#4084)
  Issue - 3697 - GitHub analyzer panic (trufflesecurity#4113)
  fix(test/snowflake): Snowflake flaky pattern test (trufflesecurity#4083)
  Netlify Analyzer (trufflesecurity#4106)
  Add xAI detector (trufflesecurity#4117)
  added tunnel authtoken detection in ngrok detector (trufflesecurity#4115)
  Postman increase http client timeout (trufflesecurity#4086)
  feat: bing subscription key support (trufflesecurity#4092)
  Improve precommit hook setup and support both global and repo pre-commit hooks, frameworkless and with pre-commit or husky (trufflesecurity#4098)
  Make a first pass at some structural introduction docs (trufflesecurity#4076)
  Update action.yml (trufflesecurity#4108)
  Refactor Netlify Detector (trufflesecurity#4102)
  chore: added github action custom detector name (trufflesecurity#3417)
  Fixed PubnubsubsciptionKey detector verification (trufflesecurity#4107)
  ...

# Conflicts:
#	pkg/engine/defaults/defaults.go
#	pkg/pb/detectorspb/detectors.pb.go
#	proto/detectors.proto
@abmussani abmussani merged commit ce126fa into trufflesecurity:main May 9, 2025
13 checks passed
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.

3 participants