Skip to content

Convert web session and SAML service provider cache collections#54478

Merged
rosstimothy merged 1 commit intomasterfrom
tross/cache_saml
May 2, 2025
Merged

Convert web session and SAML service provider cache collections#54478
rosstimothy merged 1 commit intomasterfrom
tross/cache_saml

Conversation

@rosstimothy
Copy link
Copy Markdown
Contributor

Moves SAMLIdPServiceProviders, AppSessions, WebSession, SAMLSessions, and SnowflakeSessions to the new cache collection scheme that was introduced in #52210. No additional functionality changes have been made here. This should be a purely mechanical translation to the new internal caching machinery.

@rosstimothy rosstimothy added the no-changelog Indicates that a PR does not require a changelog entry label May 2, 2025
@rosstimothy rosstimothy force-pushed the tross/cache_saml branch 2 times, most recently from 6d75c34 to 73cced9 Compare May 2, 2025 14:11
@rosstimothy rosstimothy marked this pull request as ready for review May 2, 2025 14:30
@rosstimothy rosstimothy requested a review from fspmarshall May 2, 2025 14:30
@github-actions github-actions Bot requested review from espadolini and vapopov May 2, 2025 14:30
@rosstimothy rosstimothy requested a review from avatus May 2, 2025 16:50
Moves SAMLIdPServiceProviders, AppSessions, WebSession, SAMLSessions,
and SnowflakeSessions to the new cache collection scheme that was
introduced in #52210. No additional functionality changes have been
made here. This should be a purely mechanical translation to the
new internal caching machinery.
@rosstimothy rosstimothy enabled auto-merge May 2, 2025 16:57
@rosstimothy rosstimothy added this pull request to the merge queue May 2, 2025
Merged via the queue into master with commit ee4bed8 May 2, 2025
41 checks passed
@rosstimothy rosstimothy deleted the tross/cache_saml branch May 2, 2025 17:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-changelog Indicates that a PR does not require a changelog entry size/md

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants