Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

NOISSUE - Add Redis Cache For SpiceDB #2239

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

rodneyosodo
Copy link
Member

What type of PR is this?

This is a feature PR because it adds policy caching to Redis to make auth calls faster

What does this do?

Cache policies when the first authorization request happens and delete them when removing policies

Which issue(s) does this PR fix/relate to?

No issue

Have you included tests for your changes?

Yes, I have included tests for my changes.

Did you document any new/modified feature?

Yes, I have updated the documentation for the new feature.

Notes

@rodneyosodo rodneyosodo force-pushed the noissue-cache-policies branch 9 times, most recently from 74e7614 to 2d7a24d Compare May 16, 2024 09:33
@rodneyosodo rodneyosodo marked this pull request as ready for review May 16, 2024 09:33
@rodneyosodo rodneyosodo force-pushed the noissue-cache-policies branch 5 times, most recently from 8cc4456 to faff4a4 Compare May 24, 2024 08:26
@rodneyosodo rodneyosodo added this to the S5 milestone May 28, 2024
@rodneyosodo rodneyosodo force-pushed the noissue-cache-policies branch 6 times, most recently from 2832853 to 9458c32 Compare June 12, 2024 10:00
Cache policies when the first authorization request happens and delete them when removing policies

Signed-off-by: Rodney Osodo <28790446+rodneyosodo@users.noreply.github.com>
Signed-off-by: Rodney Osodo <28790446+rodneyosodo@users.noreply.github.com>
@dborovcanin dborovcanin modified the milestones: S5, 0.15.0 Jun 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 🩺 Review and testing
Development

Successfully merging this pull request may close these issues.

None yet

2 participants