Skip to content

Backlog/v11 modules config comunication - #2584

Open
AlexSanchez-bit wants to merge 2 commits into
v11from
backlog/v11_modules_config_comunication
Open

AlexSanchez-bit wants to merge 2 commits into
v11from
backlog/v11_modules_config_comunication

Conversation

@AlexSanchez-bit

Copy link
Copy Markdown
Contributor

Main changes

  • on sensitive fields update a validator of the module config dto was sending plaintext fields causing an error on modules config plugin
  • modules config plugin was ignoring cipher errors and ignoring sensitive fields that arrives as plaintext succeeding the update but leaving an error log

@AlexSanchez-bit
AlexSanchez-bit requested a review from a team September 15, 2026 19:59
@AlexSanchez-bit AlexSanchez-bit linked an issue Sep 15, 2026 that may be closed by this pull request
3 tasks
@github-actions

Copy link
Copy Markdown

❌ Go dependencies check failed

There are outdated Go dependencies, or modules that could not be inspected.
Run bash .github/scripts/go-deps.sh --update --discover locally and
commit the updated go.mod / go.sum files.

Script output
🔍 Discovered 25 Go projects

📦 Dependencies with updates available:

  📁 ./plugins/soc-ai:
     - github.com/threatwinds/go-sdk: v1.1.31 → v1.1.32

  📁 ./plugins/crowdstrike:
     - github.com/threatwinds/go-sdk: v1.1.31 → v1.1.32

  📁 ./plugins/sophos:
     - github.com/threatwinds/go-sdk: v1.1.31 → v1.1.32

  📁 ./plugins/bitdefender:
     - github.com/threatwinds/go-sdk: v1.1.31 → v1.1.32

  📁 ./plugins/stats:
     - github.com/threatwinds/go-sdk: v1.1.31 → v1.1.32

  📁 ./plugins/azure:
     - github.com/threatwinds/go-sdk: v1.1.31 → v1.1.32

  📁 ./plugins/alerts:
     - github.com/threatwinds/go-sdk: v1.1.31 → v1.1.32

  📁 ./plugins/inputs:
     - github.com/threatwinds/go-sdk: v1.1.31 → v1.1.32

  📁 ./plugins/config:
     - github.com/threatwinds/go-sdk: v1.1.31 → v1.1.32

  📁 ./plugins/events:
     - github.com/threatwinds/go-sdk: v1.1.31 → v1.1.32

  📁 ./plugins/modules-config:
     - github.com/aws/aws-sdk-go-v2/config: v1.33.4 → v1.33.5
     - github.com/aws/aws-sdk-go-v2/credentials: v1.20.4 → v1.20.5
     - github.com/aws/aws-sdk-go-v2/service/cloudwatchlogs: v1.87.0 → v1.88.0
     - github.com/aws/aws-sdk-go-v2/service/sts: v1.50.0 → v1.51.0
     - github.com/threatwinds/go-sdk: v1.1.31 → v1.1.32
     - google.golang.org/api: v0.297.0 → v0.298.0

  📁 ./plugins/geolocation:
     - github.com/threatwinds/go-sdk: v1.1.31 → v1.1.32

  📁 ./plugins/o365:
     - github.com/threatwinds/go-sdk: v1.1.31 → v1.1.32

  📁 ./plugins/aws:
     - github.com/aws/aws-sdk-go-v2/config: v1.33.4 → v1.33.5
     - github.com/aws/aws-sdk-go-v2/credentials: v1.20.4 → v1.20.5
     - github.com/aws/aws-sdk-go-v2/service/cloudwatchlogs: v1.87.0 → v1.88.0
     - github.com/threatwinds/go-sdk: v1.1.31 → v1.1.32

  📁 ./plugins/gcp:
     - github.com/threatwinds/go-sdk: v1.1.31 → v1.1.32
     - google.golang.org/api: v0.297.0 → v0.298.0

  📁 ./plugins/rule-flood-guard:
     - github.com/threatwinds/go-sdk: v1.1.31 → v1.1.32

  📁 ./as400:
     - github.com/threatwinds/go-sdk: v1.1.31 → v1.1.32

  📁 ./as400/updater:
     - github.com/threatwinds/go-sdk: v1.1.31 → v1.1.32

  📁 ./installer:
     - github.com/cloudfoundry/gosigar: v1.3.126 → v1.3.127

  📁 ./agent-manager:
     - github.com/threatwinds/go-sdk: v1.1.31 → v1.1.32
     - gorm.io/driver/postgres: v1.6.2 → v1.6.3

  📁 ./utmstack-collector:
     - github.com/threatwinds/go-sdk: v1.1.31 → v1.1.32

  📁 ./agent:
     - github.com/threatwinds/go-sdk: v1.1.31 → v1.1.32

�[0;31m❌ Please update dependencies before merging.�[0m

@github-actions

Copy link
Copy Markdown

🛑 AI review — Blocking issues

One or more high/critical issues can break things and must be fixed before merging. Details below.

🛑 architecture (gemini-3-flash-lite) — blocking — must fix before merge

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

  • high (n/a):0 — ThreatWinds API returned HTTP 429 (fail-safe: a review that cannot run is treated as blocking).

🛑 bugs (gemini-3-flash-lite) — blocking — must fix before merge

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

  • high (n/a):0 — ThreatWinds API returned HTTP 429 (fail-safe: a review that cannot run is treated as blocking).

🛑 security (gemini-3-flash-lite) — blocking — must fix before merge

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

  • high (n/a):0 — ThreatWinds API returned HTTP 429 (fail-safe: a review that cannot run is treated as blocking).

@utmstackprapprover utmstackprapprover Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes requested — Go dependencies check failed (see above).

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.

v11 encription error between backend and modules config

1 participant