Refresh JWKS and RBACPolicy in background #135
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
jhaven-stytch
approved these changes
Nov 7, 2025
Contributor
jhaven-stytch
left a comment
jhaven-stytch
left a comment
There was a problem hiding this comment.
I would say my notes only need addressing IF you anticipate the refresh job being cancelled/cancellable. From what I can tell, since we use a result class for the network call, the job shouldn't get cancelled from any failures, so isActive should always be true, but...
logan-stytch
force-pushed
the
logan/back-5074
branch
from
944e814 to
ede1f9b
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR implements background refresh for both JWKS and RBAC policy caches to reduce latency. '
The RBAC PolicyCache now uses a 1-hour TTL (up from 5 minutes) with coroutine-based background refresh that proactively updates the policy every hour. We're standardizing on 1-hour TTL for our caches across the Stytch SDKs.
For JWKS caching, a new JwksCache factory helper was created that configures jose4j's HttpsJwks with a 1-hour cache duration and 5-minute refresh threshold for automatic background refresh.