ieb/sparsemapcontent — Issue #136: Race Condition reported with AuthorizableManagerImpl cache
Author: ieb
Created: 2011-12-02 23:57:50+00:00
State: open

At https://github.com/sakaiproject/nakamura/pull/462 there was a race condition reported in the AuthorizableManagerImpl cache.

The solution used there was to create a new admin session to confirm that the delete was performed. I havent looked into it in detail but opening a new session will break any transactional integrity (when that works is completed), and the admin session may bypass access control. So the solution is really a sicking paster and will probably result in other problems especially where there are synchronous listeners attached. If it is a race condition in the cache, thats what should be fixed.

Unfortunately there are few detailed on the pull request, KERN about the suspected race condition in AuthorizableManagerImpl, and its not clear which version of the code is being used. There was a fix in this area about 2 months ago.

Recording here until I get a chance to see if the issue is reproducible in the current code base or something to do with the forked version being used by Nakamura.