Skip to content

Conversation

@burythehammer
Copy link
Collaborator

@burythehammer burythehammer commented Nov 11, 2025

Fixed

  • rediscloud_active_active_subscription_database: Fixed drift detection for enable_default_user in region overrides. Regions now correctly inherit from global_enable_default_user when not explicitly set, eliminating spurious diffs.
  • rediscloud_active_active_subscription_database: Fixed drift detection for global_data_persistence. This field now correctly tracks API-returned values without causing unexpected diffs.
  • rediscloud_subscription and rediscloud_active_active_subscription: customer_managed_key_enabled and customer_managed_key_deletion_grace_period fields now support computed values from the API.

@burythehammer burythehammer changed the title fix: Enable default user override fix: default user override Nov 11, 2025
Copy link

@jit-ci jit-ci bot left a comment

Choose a reason for hiding this comment

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

❌ Jit has detected 1 important finding in this PR that you should review.
The finding is detailed below as a comment.
It’s highly recommended that you fix this security issue before merge.

Repository Risks:

  • Database Integration: Connects to a database, often involving sensitive data that must be securely managed.
  • Internally Accessible: Accessible only within the internal network, reducing exposure to external threats but still requiring proper controls.

Repository Context:

graph LR
    GitHub$Repository_U23_RedisLabs/terraform_U2D_provider_U2D_rediscloud["GitHub Repository<br/>RedisLabs/terraform-provider-rediscloud"]:::GitHub$Repository
    Team_U23_SecurityManagers["Team<br/>SecurityManagers"]:::Team
    DBIntegration_U23_redis["DBIntegration<br/>redis"]:::DBIntegration
    Team_U23_SecurityManagers -- "Owns" --> GitHub$Repository_U23_RedisLabs/terraform_U2D_provider_U2D_rediscloud
    GitHub$Repository_U23_RedisLabs/terraform_U2D_provider_U2D_rediscloud -- "Is accessible to" --> DBIntegration_U23_redis
Loading

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