Skip to content

feat: additional findings processor - #7518

Open
lcampbell2 wants to merge 15 commits into
masterfrom
feat/additional-findings-processor
Open

feat: additional findings processor#7518
lcampbell2 wants to merge 15 commits into
masterfrom
feat/additional-findings-processor

Conversation

@lcampbell2

Copy link
Copy Markdown
Collaborator

Adds a new scanners/findings-processor Go service that consumes finding events from NATS JetStream and upserts normalized documents into ArangoDB additionalFindings.

  • Introduces a standalone service scaffold (cmd/service, internal/*, go.mod, Dockerfile, Makefile, cloudbuild, env template, README).
  • Implements config/env loading, structured logging, and graceful shutdown.
  • Subscribes to scans.findings.* (configurable), validates event payloads, and applies explicit Ack/Nak/Term handling.
  • Adds finding normalization + deterministic key generation, then upsert logic that creates new docs or updates lastSeen and occurrence count for existing docs.
  • Includes broad unit test coverage across config, model validation/mapping, DB upsert flow, and runner/event behavior.

@lcampbell2
lcampbell2 requested a review from FestiveKyle August 14, 2026 17:05
@lcampbell2

Copy link
Copy Markdown
Collaborator Author

/gcbrun

@lcampbell2

Copy link
Copy Markdown
Collaborator Author

/gcbrun(84c5a2c)

@lcampbell2
lcampbell2 marked this pull request as ready for review August 14, 2026 17:22
Comment thread scanners/findings-processor/internal/database/upsert.go Outdated
Comment thread scanners/findings-processor/internal/database/upsert.go Outdated
Comment thread scanners/findings-processor/internal/model/finding.go
Comment thread scanners/findings-processor/internal/runner/actions.go Outdated
Comment thread scanners/findings-processor/internal/database/upsert_test.go Outdated
Comment thread scanners/findings-processor/internal/database/upsert.go Outdated
Comment thread scanners/findings-processor/README.md Outdated
Comment thread scanners/findings-processor/internal/runner/deps.go Outdated
@lcampbell2
lcampbell2 requested a review from FestiveKyle August 31, 2026 17:51
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