Opened 10 years ago

Last modified 17 months ago

#2917 assigned defect

cache_size being limited to 25

Reported by: John A. Barbuto Owned by: Emmanuel Blot
Priority: normal Component: LdapPlugin
Severity: normal Keywords:
Cc: Trac Release: 0.10



While troubleshooting a performance issue, I found that the LDAP cache_size can't go higher than 25, despite what is set in trac.ini. Here's the relevant code in lines 71 and 187 of

self._cache_size = min(25, int(self.config.get('ldap', 'cache_size', '100')))
self._cache_size = min(25, int(cache_size))

Why have a default of 100 for cache_size when it's being forced down to 25? A limit for sanity checking makes sense, but I think it should be much higher. Fixing this in our installation made our Trac significantly faster.

Attachments (0)

Change History (2)

comment:1 Changed 10 years ago by Emmanuel Blot

Status: newassigned

Very true.

I have used the LdapPlugin for a very long time, and it definitely needs some improvements and fixes.

As I plan to use again this plugin in our environment, I hope to have some time to work on it.

comment:2 Changed 9 years ago by Jodok Batlogg

it should say max(...)instead of min(...)

Modify Ticket

Change Properties
Set your email in Preferences
as assigned The owner will remain Emmanuel Blot.

Add Comment

E-mail address and name can be saved in the Preferences.

Note: See TracTickets for help on using tickets.