openwallet-foundation / acapy-vc-authn-oidc / 22501146218

91%

Build Type

push

github

Committed by web-flow

Commit Message

Redis cluster sentinel (#968)

* add support for sentinel and clustered redis

Signed-off-by: Gavin Jaeger-Freeborn <gavinfreeborn@gmail.com>

* apply linting

Signed-off-by: Gavin Jaeger-Freeborn <gavinfreeborn@gmail.com>

* added missing redis_utils.py

Signed-off-by: Gavin Jaeger-Freeborn <gavinfreeborn@gmail.com>

* Refactor Redis configuration handling for improved clarity and compatibility

- Update environment variable parsing to require REDIS_HOST in host:port format, enhancing clarity and reducing legacy issues.
- Remove the deprecated REDIS_PORT variable; consolidate port information within REDIS_HOST for consistency.
- Streamline validation and normalization of Redis settings, ensuring errors are raised early and precisely.
- Adjust documentation and comments to reflect new configuration requirements and usage patterns.
- Update tests to cover new behaviors and ensure backward compatibility.

Signed-off-by: Gavin Jaeger-Freeborn <gavinfreeborn@gmail.com>

* Add normalization for Redis configuration at import time

Signed-off-by: Gavin Jaeger-Freeborn <gavinfreeborn@gmail.com>

* Refactor `socketio.py`: Remove duplicate import of `Sentinel`

Signed-off-by: Gavin Jaeger-Freeborn <gavinfreeborn@gmail.com>

* Format with black

Signed-off-by: Gavin Jaeger-Freeborn <gavinfreeborn@gmail.com>

* Fix broken cleanup tests

Signed-off-by: Gavin Jaeger-Freeborn <gavinfreeborn@gmail.com>

* Add tests for Redis utility functions and socket manager

This commit introduces unit tests for Redis utility functions and
enhances error handling in the socket manager related to Sentinel
connectivity. These changes improve the stability and reliability of
the Redis implementation used in the application.

Signed-off-by: Gavin Jaeger-Freeborn <gavinfreeborn@gmail.com>

* Refactor Redis tests for password handling and sentinel host parsing

- Updated tests to reflect that REDIS_PASSWORD is used for both sentinel and master authentication, simplifying configurations... (continued)

Run Details

296 of 335 new or added lines in 11 files covered. (88.36%)

3 existing lines in 2 files now uncovered.

2461 of 2764 relevant lines covered (89.04%)

0.89 hits per line