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

89%

push

github

Redis based PyOP wrapper implementation (#878)

* Initial implementation of redis wrapper

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

* Multi pod focused changes enhancing OIDC Authentication and Client Management

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

- Added `pyop_user_id` field to `AuthSessionBase` model to store original user ID for authorization code regeneration.
- Updated `ClientConfigurationCRUD` to remove unnecessary provider reloads, leveraging `DynamicClientDatabase` for on-demand MongoDB access.
- Implemented `DynamicClientDatabase` class to cache client configurations and ensure consistency across pods.
- Refined `generate_auth_code` function to ensure the use of the correct user ID from the `AuthSession` during code regeneration.
- Added handling for legacy sessions missing `pyop_user_id`, generating new UUIDs when necessary.
- Improved comments and documentation throughout the code for clarity on changes and their purposes.

* Allow swapping between stateless and redis storage

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

- Introduced conditional logic to select between `RedisWrapperWithPack` for multi-pod deployments and `StatelessWrapper` for single-pod deployments based on the `USE_REDIS_ADAPTER` setting.
- Updated storage initialization to improve clarity and maintainability.
- Added logging to reflect the chosen storage backend and warn about stateless deployment limitations.

* Added redis wrapper test

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

* attempt to fix python 3.12 errors

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

* dropped test dependant on time

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

* More tests

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

* self.collection -> self._collection and recover subject identifier

Signed-off-by: Gavin Jaeger-Freeborn <gavinfreeborn@gmail.co... (continued)

69 of 87 new or added lines in 2 files covered. (79.31%)

1577 of 1707 relevant lines covered (92.38%)

0.92 hits per line