Skip to content

PSMDB-1947-8.0-LDAP-User-Cache-Refresh-options#1116

Merged
rasika-chivate merged 4 commits into8.0from
PSMDB-1947-8.0-LDAP-User-Cache-Refresh-options
Apr 1, 2026
Merged

PSMDB-1947-8.0-LDAP-User-Cache-Refresh-options#1116
rasika-chivate merged 4 commits into8.0from
PSMDB-1947-8.0-LDAP-User-Cache-Refresh-options

Conversation

@rasika-chivate
Copy link
Copy Markdown
Collaborator

@rasika-chivate rasika-chivate commented Mar 27, 2026

  • Understand feedback: add runtime/CLI/config examples for ldapUserCacheInvalidationInterval
  • Add examples for ldapUserCacheInvalidationInterval in docs/authorization.md (both interval-based refresh and expiration-based invalidation strategies shown)
  • Final code review — no issues found

⌨️ Start Copilot coding agent tasks without leaving your editor — available in VS Code, Visual Studio, JetBrains IDEs and Eclipse.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds documentation for new LDAP user cache refresh behavior, helping operators tune authentication performance by controlling when cached $external user entries are refreshed vs. invalidated.

Changes:

  • Document new LDAP cache refresh options (ldapUserCacheRefreshInterval, ldapShouldRefreshUserCacheEntries) and how they change cache behavior.
  • Describe interaction with the existing invalidation/eviction interval (ldapUserCacheInvalidationInterval).
  • Provide configuration examples for runtime, command-line, and config-file usage (for the new options).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docs/authorization.md
Comment thread docs/authorization.md
Comment on lines +111 to +112
- `ldapUserCacheInvalidationInterval` controls how long (in seconds) cached LDAP user entries remain valid before they expire and are evicted from the cache. If you do not set this parameter explicitly, Percona Server for MongoDB uses the built-in default for your version. This parameter applies when `ldapShouldRefreshUserCacheEntries` is set to `false`.

Copy link

Copilot AI Mar 27, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This section documents ldapUserCacheInvalidationInterval, but the examples below only show how to set ldapUserCacheRefreshInterval / ldapShouldRefreshUserCacheEntries. Consider adding a matching runtime/CLI/config example for ldapUserCacheInvalidationInterval (or explicitly stating where/how to configure it) so readers can apply the expiration-based strategy as described.

Copilot uses AI. Check for mistakes.
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot apply changes based on this feedback

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added Runtime, Command line, and Configuration file examples for ldapUserCacheInvalidationInterval (expiration-based strategy) alongside the existing examples for the interval-based refresh strategy (commit cdfd1c7). Both strategies now have matching configuration examples under their own labeled sections.

…sh examples

Agent-Logs-Url: https://github.com/percona/psmdb-docs/sessions/f13f0760-9be9-446a-be43-95b9cdbc9289

Co-authored-by: rasika-chivate <95711051+rasika-chivate@users.noreply.github.com>
Comment thread docs/authorization.md Outdated
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docs/authorization.md Outdated
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@nastena1606 nastena1606 temporarily deployed to PSMDB-1947-8.0-LDAP-User-Cache-Refresh-options - psmdb-docs-8.0 PR #1116 March 28, 2026 14:49 — with Render Destroyed
@rasika-chivate rasika-chivate merged commit 3b5ae11 into 8.0 Apr 1, 2026
@rasika-chivate rasika-chivate deleted the PSMDB-1947-8.0-LDAP-User-Cache-Refresh-options branch April 1, 2026 11:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants