This just came in:

The patch seems to fix the issue, however it's missing tests -- actually the cppunit tests aren't testing any acl functionality, this (missing tests) is a major oversight we need to address asap (post 3.1)!

There is a workaround for this issue - use the async version of the method. Seems that no one ever used this function before (or never reported it at least) because it's an obvious problem and will always fail.

I'd like to hold off on fixing this until post 3.1 (3.1.0 or 3.2 whichever comes first) where we can also address the testing issue for acls. There is a workaround -- Chris you ok with that? What do the rest of you think (vote).


Reply via email to