Modify

Opened 7 years ago

Last modified 6 years ago

#2917 assigned defect

cache_size being limited to 25

Reported by: jab Owned by: eblot
Priority: normal Component: LdapPlugin
Severity: normal Keywords:
Cc: Trac Release: 0.10

Description

Hi,

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 api.py:

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 7 years ago by eblot

  • Status changed from new to assigned

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 6 years ago by jodok

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

Add Comment

Modify Ticket

Action
as assigned The owner will remain eblot.
Author


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

 
Note: See TracTickets for help on using tickets.