Opened 10 years ago

Last modified 21 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 10 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.