Skip to content

fix[backend](): wired new bulk operations on server and modules startup - #2457

Merged
AlexSanchez-bit merged 1 commit into
release/v12.0.0from
backlog/v12_backend_wiring
Aug 12, 2026
Merged

fix[backend](): wired new bulk operations on server and modules startup#2457
AlexSanchez-bit merged 1 commit into
release/v12.0.0from
backlog/v12_backend_wiring

Conversation

@AlexSanchez-bit

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions

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 (gemini-3-flash-lite) — minor findings

Summary: Introduce tenant-lister closures into module initialization and route registration.

  • medium backend/modules.go:135 — Duplicated tenant listing closure logic. Consider extracting this function into a shared helper inside the tenant module or pkg/ to avoid copy-pasting closure initialization across modules.

🛑 bugs (gemini-3-flash-lite) — high/critical — please review

Summary: Potential nil pointer dereference in tenantLister when invoked before tenantMod initialization

  • high backend/modules.go:137 — tenantMod is nil when tenantLister is called. If tenantLister is invoked before tenantMod is assigned on line 217, it will result in a nil pointer dereference (panic: runtime error: invalid memory address or nil pointer dereference) when calling tenantMod.GetTenantUsecase().

security (gemini-3-flash-lite) — clean

Summary: Wiring up tenant lister closures and platform middleware across backend modules without introducing vulnerabilities.

No findings.

🔴 go-deps — pending updates

🔍 Discovered 30 Go projects

📦 Dependencies with updates available:

  📁 ./plugins/events:
     - github.com/threatwinds/go-sdk: v1.1.27-0.20260811073440-251cb9d842cd → v1.1.28

  📁 ./plugins/rule-flood-guard:
     - github.com/threatwinds/go-sdk: v1.1.27-0.20260811073440-251cb9d842cd → v1.1.28

  📁 ./plugins/stats:
     - github.com/threatwinds/go-sdk: v1.1.27-0.20260811073440-251cb9d842cd → v1.1.28

  📁 ./plugins/feeds:
     - github.com/threatwinds/go-sdk: v1.1.27-0.20260811073440-251cb9d842cd → v1.1.28

  📁 ./plugins/alerts:
     - github.com/threatwinds/go-sdk: v1.1.27-0.20260811073440-251cb9d842cd → v1.1.28

  📁 ./plugins/aws:
     - github.com/threatwinds/go-sdk: v1.1.27-0.20260811073440-251cb9d842cd → v1.1.28

  📁 ./backend:
     - github.com/threatwinds/go-sdk: v1.1.27-0.20260811073440-251cb9d842cd → v1.1.28

❌ Please update dependencies before merging.

@AlexSanchez-bit
AlexSanchez-bit merged commit 9cd899c into release/v12.0.0 Aug 12, 2026
1 check passed
@AlexSanchez-bit
AlexSanchez-bit deleted the backlog/v12_backend_wiring branch August 12, 2026 21:38
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.

1 participant