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.

Sign up for GitHub

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

Jump to bottom

Update SumoLogic detector #3511

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Update SumoLogic detector #3511

wants to merge 1 commit into from

Conversation

rgmz
Copy link
Contributor

@rgmz rgmz commented Oct 25, 2024 •
edited
Loading

Description:

This makes a number of fixes and improvements to the SumoLogic detector.

  • Use tri-state verification
  • Update keywords
  • Improve id pattern
  • Deduplicate matches
  • Add URL pattern to detect region-specific API endpoints
  • Adds unit tests

Checklist:

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

@rgmz rgmz force-pushed the feat/update-sumologic branch 4 times, most recently from b9069f2 to d8f87f2 Compare October 26, 2024 16:33
@rgmz rgmz marked this pull request as ready for review October 26, 2024 16:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
hacktoberfest-2024
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@rgmz @zricethezav