Skip to content

feat(parsing-filters): enhance kv step handling with fieldSplit norma… - #3011

Merged
AlexSanchez-bit merged 4 commits into
release/v12.0.0from
backlog/kv-step-fieldSplit
Oct 29, 2026
Merged

AlexSanchez-bit merged 4 commits into
release/v12.0.0from
backlog/kv-step-fieldSplit

Conversation

@AndresTK89

Copy link
Copy Markdown

enhance kv step handling with fieldSplit normalization and add tests

@github-actions

github-actions Bot commented Oct 29, 2026 •

Copy link
Copy Markdown

🛑 AI review — High/critical findings

One or more high/critical issues were found. Please review and fix before merging if they're real.

⚠️ architecture (silas-1.7-pro) — minor findings

Summary: Frontend parsing-filter UI/i18n changes; minor contract drift by injecting default kv fieldSplit in serializeFilter, no agent/critical path impact.

  • low frontend/src/features/parsing-filters/lib/filter-model.ts:173 — serializeFilter now injects a default kv fieldSplit in the frontend, changing the YAML contract with the Event Processor. Ensure old/new Event Processor versions treat explicit ' ' identically to missing; otherwise move the default to the shared/backend parsing contract.

🛑 bugs (silas-1.7-pro) — high/critical — please review

Summary: AI review could not parse model response — manual review recommended.

  • high (n/a):0 — ThreatWinds API returned HTTP 503 (fail-safe fallback — flagged for manual attention, does not block the merge).

✅ security (silas-1.7-pro) — clean

Summary: No new vulnerabilities or customer-facing information disclosures identified in the diff.

No findings.

🔴 go-deps — pending updates

🔍 Discovered 31 Go projects

📦 Dependencies with updates available:

  📁 ./agent:
     - github.com/threatwinds/go-sdk: v1.1.28 → v1.1.38
     - google.golang.org/grpc: v1.84.0 → v1.85.0

  📁 ./tools/rulecheck:
     - github.com/threatwinds/go-sdk: v1.1.31 → v1.1.38

  📁 ./agent-manager:
     - github.com/threatwinds/go-sdk: v1.1.31 → v1.1.38
     - google.golang.org/grpc: v1.84.0 → v1.85.0

  📁 ./collectors/forwarder:
     - github.com/threatwinds/go-sdk: v1.1.31 → v1.1.38
     - google.golang.org/grpc: v1.84.0 → v1.85.0

  📁 ./collectors/as400:
     - github.com/threatwinds/go-sdk: v1.1.31 → v1.1.38
     - google.golang.org/grpc: v1.84.0 → v1.85.0

  📁 ./collectors/utmstack:
     - github.com/threatwinds/go-sdk: v1.1.31 → v1.1.38
     - google.golang.org/grpc: v1.84.0 → v1.85.0

  📁 ./backend:
     - github.com/aws/aws-sdk-go-v2/config: v1.33.11 → v1.33.12
     - github.com/aws/aws-sdk-go-v2/credentials: v1.20.11 → v1.20.12
     - github.com/aws/aws-sdk-go-v2/service/sts: v1.51.5 → v1.51.6
     - github.com/crowdstrike/gofalcon: v0.23.0 → v0.24.0
     - google.golang.org/api: v0.309.0 → v0.310.0
     - google.golang.org/grpc: v1.84.0 → v1.85.0

  📁 ./plugins/aws:
     - github.com/aws/aws-sdk-go-v2: v1.47.4 → v1.47.5
     - github.com/aws/aws-sdk-go-v2/config: v1.33.11 → v1.33.12
     - github.com/aws/aws-sdk-go-v2/credentials: v1.20.11 → v1.20.12
     - github.com/aws/aws-sdk-go-v2/service/cloudwatchlogs: v1.90.2 → v1.90.3

  📁 ./plugins/gcp:
     - google.golang.org/api: v0.309.0 → v0.310.0

  📁 ./plugins/crowdstrike:
     - github.com/crowdstrike/gofalcon: v0.23.0 → v0.24.0

  📁 ./log-input:
     - google.golang.org/grpc: v1.84.0 → v1.85.0

❌ Please update dependencies before merging.

@AlexSanchez-bit
AlexSanchez-bit merged commit 9990302 into release/v12.0.0 Oct 29, 2026
1 check passed
@AlexSanchez-bit
AlexSanchez-bit deleted the backlog/kv-step-fieldSplit branch October 29, 2026 21:02
@AlexSanchez-bit
AlexSanchez-bit restored the backlog/kv-step-fieldSplit branch October 29, 2026 21:04
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.

2 participants