Skip to content

fix[backend](integrations): removed aws log group in aws integrations module#2338

Merged
AlexSanchez-bit merged 2 commits into
release/v12.0.0from
backlog/v12_aws_integrations
Jul 9, 2026
Merged

fix[backend](integrations): removed aws log group in aws integrations module#2338
AlexSanchez-bit merged 2 commits into
release/v12.0.0from
backlog/v12_aws_integrations

Conversation

@AlexSanchez-bit

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions

github-actions Bot commented Jul 9, 2026

Copy link
Copy Markdown

🛑 AI review — Sensitive area, extra care recommended

This PR touches critical paths or introduces changes the model cannot judge with sufficient confidence. Review carefully before merging.

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

Summary: Removal of AWS CloudWatch integration logic breaks existing user configurations and integration contracts.

  • high backend/modules/integrations/repository/schema_provider.go:15 — Removal of 'aws_log_group_name' from schema breaks existing integrations. This is a breaking change for users relying on CloudWatch log ingestion.
  • high backend/modules/integrations/verifier/verifier_aws.go:26 — Removal of CloudWatch verification logic constitutes a breaking change in the integration contract. Existing AWS integrations configured for CloudWatch will fail or become invalid.

⚠️ bugs (gemini-3-flash-lite) — minor findings

Summary: Removal of AWS CloudWatch log group validation logic and dependency updates.

  • medium backend/modules/integrations/verifier/verifier_aws.go:13 — The function verifyAWSIAMUser still requires 'aws_log_group_name' in the validation check, but the variable is no longer used in the function body. This will cause validation errors for users who do not provide this field, even though the backend no longer consumes it.

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

Summary: Removal of AWS CloudWatch log group verification logic and routine dependency updates.

No findings.

🟢 go-deps — up to date

No pending Go dependency updates.

@AlexSanchez-bit AlexSanchez-bit linked an issue Jul 9, 2026 that may be closed by this pull request
@AlexSanchez-bit

Copy link
Copy Markdown
Contributor Author

log group name is unused and will no break cloudwatch logic

@AlexSanchez-bit AlexSanchez-bit merged commit b85427b into release/v12.0.0 Jul 9, 2026
1 check passed
@AlexSanchez-bit AlexSanchez-bit deleted the backlog/v12_aws_integrations branch July 9, 2026 19:40
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.

v12 unused field on aws integration guide

1 participant