From fa0a82a92a986b8b0beb612a6833d028306c170b Mon Sep 17 00:00:00 2001 From: Carlo Costino Date: Tue, 7 Oct 2025 15:11:32 -0400 Subject: [PATCH 1/2] Remove secure Redis protocol workaround This changeset removes a workaround that has been in place to account for a small error in the Cloud.gov AWS broker that has recently been fixed. h/t to @markdboyd for the fix here! https://github.com/cloud-gov/aws-broker/pull/440 Signed-off-by: Carlo Costino --- app/cloudfoundry_config.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/cloudfoundry_config.py b/app/cloudfoundry_config.py index b79d831c2..79eb75b5d 100644 --- a/app/cloudfoundry_config.py +++ b/app/cloudfoundry_config.py @@ -19,7 +19,7 @@ class CloudfoundryConfig: try: return self.parsed_services["aws-elasticache-redis"][0]["credentials"][ "uri" - ].replace("redis://", "rediss://") + ] except KeyError: return os.environ.get("REDIS_URL") From c24e057311caa1759abcf36fb15f278ee4aa6836 Mon Sep 17 00:00:00 2001 From: Carlo Costino Date: Tue, 7 Oct 2025 15:29:47 -0400 Subject: [PATCH 2/2] Fix test fixture Signed-off-by: Carlo Costino --- tests/app/test_cloudfoundry_config.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/app/test_cloudfoundry_config.py b/tests/app/test_cloudfoundry_config.py index 433f2c73e..10ee8ea44 100644 --- a/tests/app/test_cloudfoundry_config.py +++ b/tests/app/test_cloudfoundry_config.py @@ -16,7 +16,7 @@ bucket_credentials = { @pytest.fixture def vcap_services(): return { - "aws-elasticache-redis": [{"credentials": {"uri": "redis://xxx:6379"}}], + "aws-elasticache-redis": [{"credentials": {"uri": "rediss://xxx:6379"}}], "s3": [ { "name": "notify-api-csv-upload-bucket-test",