openwallet-foundation / acapy-vc-authn-oidc
92%

DEFAULT BRANCH: main
Repo Added
Files 31
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

LAST BUILD ON BRANCH main
branch: main
CHANGE BRANCH
x

coverage: 92.384% (+0.5%) from 91.885%
18732506905

push

github

web-flow 
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

Relevant lines Covered
1707 RELEVANT LINES 1577 COVERED LINES
0.92 HITS PER LINE
Source Files on main

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
18732506905 main 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-... push web-flow github
92.38
18390534355 main Handle Redis Connection Failure (#856) * Better handle redis failing and working without redis Signed-off-by: Gavin Jaeger-Freeborn <gavinfreeborn@gmail.com> * Remove REDIS_REQUIRED and instead default to crashing on error connecting to redis ... push web-flow github
91.89
17839097595 main Chore(chart): delete chart files and add chart relocation notice (#864) * chore: delete helm chart files Signed-off-by: Ivan P <2119240+i5okie@users.noreply.github.com> * chore: update readme with relocation notice Signed-off-by: Ivan P <21192... push web-flow github
92.11
17626887472 main Merge pull request #855 from i5okie/feat/chart-update [chart] Update Acapy sub-chart to version 0.2.1 to use new secrets values push web-flow github
92.11
17590361276 main Merge pull request #853 from openwallet-foundation/redis-disable Default to disabling Redis push web-flow github
92.11
17565237324 main Add Redis for websocket management across multiple pods (#825) * Add Redis support for multi-pod Socket.IO communication and scale controller - Updated README.md to include new commands for single pod and scaling controller. - Modified docker-co... push web-flow github
92.11
17440121130 main [chart] fix: restart triggering annotation should be a pod annotation (#841) * move restart trigger annotation to pod metadata Signed-off-by: Ivan P <2119240+i5okie@users.noreply.github.com> * bump chart version, update readme Signed-off-by: I... push web-flow github
91.5
17329242877 main [chart] hot-fix: deployment annotation (#840) * fix deployment annotation Signed-off-by: Ivan P <2119240+i5okie@users.noreply.github.com> * use latest acapy release 0.1.9 Signed-off-by: Ivan P <2119240+i5okie@users.noreply.github.com> * bump ... push web-flow github
91.5
17311071010 main add owf helm repo, and perform repo update (#838) Signed-off-by: Ivan P <2119240+i5okie@users.noreply.github.com> Co-authored-by: Emiliano Suñé <emiliano.sune@gmail.com> push web-flow github
91.5
17311061336 main add annotation to force rollout restart on each deploy (#839) Signed-off-by: Ivan P <2119240+i5okie@users.noreply.github.com> push web-flow github
91.5
See All Builds (388)