Skip to content

[NDB_Client] Remove guard on session_cache_limiter#9980

Open
driusan wants to merge 1 commit intoaces:mainfrom
driusan:PrivateSessionCache
Open

[NDB_Client] Remove guard on session_cache_limiter#9980
driusan wants to merge 1 commit intoaces:mainfrom
driusan:PrivateSessionCache

Conversation

@driusan
Copy link
Copy Markdown
Collaborator

@driusan driusan commented Aug 26, 2025

All logged in user session data is private. There is no reason to restrict the session_cache_limiter to the API. (In fact, it is a bad idea.)

@driusan driusan added Category: Bug PR or issue that aims to report or fix a bug Category: Security PR or issue that aims to improve security Difficulty: Trivial PR or issue whose implementation is obviously correct and no-risk (typo, formatting...) labels Aug 26, 2025
@github-actions github-actions Bot added the Language: PHP PR or issue that update PHP code label Aug 26, 2025
@driusan driusan force-pushed the PrivateSessionCache branch from 6d2d41f to e528d00 Compare August 29, 2025 14:10
@driusan driusan force-pushed the PrivateSessionCache branch from e528d00 to c21e3b6 Compare September 8, 2025 18:24
@driusan driusan self-assigned this Sep 8, 2025
@driusan driusan force-pushed the PrivateSessionCache branch from c21e3b6 to 4e1caf0 Compare February 9, 2026 15:55
@driusan driusan force-pushed the PrivateSessionCache branch from 4e1caf0 to 7619987 Compare April 13, 2026 20:31
All logged in user session data is private. There is no reason to restrict
the session_cache_limiter to the API. (In fact, it is a bad idea.)
@driusan driusan force-pushed the PrivateSessionCache branch from 7619987 to 837dcc9 Compare April 24, 2026 20:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Category: Bug PR or issue that aims to report or fix a bug Category: Security PR or issue that aims to improve security Difficulty: Trivial PR or issue whose implementation is obviously correct and no-risk (typo, formatting...) Language: PHP PR or issue that update PHP code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant