Skip to content

Fix broken regex alternation in detections#4116

Open
srkyn wants to merge 5 commits into
splunk:developfrom
srkyn:fix-broken-regex-match
Open

Fix broken regex alternation in detections#4116
srkyn wants to merge 5 commits into
splunk:developfrom
srkyn:fix-broken-regex-match

Conversation

@srkyn

@srkyn srkyn commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

Summary

Validation

  • YAML parsed successfully for all touched detections
  • Regex sanity check compiled match() patterns from the touched searches
  • git diff --check passed

Fixes #4077

@nasbench nasbench added this to the v6.1.0 milestone Jun 4, 2026
@srkyn

srkyn commented Jun 8, 2026

Copy link
Copy Markdown
Contributor Author

Rechecked the current failed CI run. The detection YAML/build checks are passing, and the two failing jobs appear blocked before exercising this regex change because the credential variables are empty in the run:

  • appinspect: APPINSPECTUSERNAME / APPINSPECTPASSWORD are empty, then Splunk API login returns 400 Client Error
  • unit-testing: contentctl-ng install exits with "Splunkbase credentials not provided in environment variables" before producing test_results/summary.yml

I will leave the branch scoped to the regex alternation fix unless you want a repo-side change.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] Broken REGEX for several detections

2 participants