You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@whimsical.apache.org by se...@apache.org on 2017/12/14 10:26:34 UTC

[whimsy] branch master updated: Revert, because that caused lots of other issues

This is an automated email from the ASF dual-hosted git repository.

sebb pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/whimsy.git


The following commit(s) were added to refs/heads/master by this push:
     new cab36fe  Revert, because that caused lots of other issues
cab36fe is described below

commit cab36fea87d7f4c711af038d9956a409031ccc20
Author: Sebb <se...@apache.org>
AuthorDate: Thu Dec 14 10:26:22 2017 +0000

    Revert, because that caused lots of other issues
    
    Currently only bind operations are affected
---
 lib/whimsy/asf/ldap.rb | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/lib/whimsy/asf/ldap.rb b/lib/whimsy/asf/ldap.rb
index 53a57bc..d8674fc 100644
--- a/lib/whimsy/asf/ldap.rb
+++ b/lib/whimsy/asf/ldap.rb
@@ -141,14 +141,14 @@ module ASF
       ASF.ldap.unbind if ASF.ldap.bound? rescue nil
       ldap = ASF.init_ldap(true)
       if block
-        ASF.flush_weakrefs
+        self.flush_weakrefs # TODO fails
         ldap.bind(dn, password, &block)
         ASF.init_ldap(true)
       else
         ldap.bind(dn, password)
       end
     ensure
-      ASF.flush_weakrefs
+      self.flush_weakrefs # TODO fails
     end
 
     # validate HTTP authorization, and optionally invoke a block bound to

-- 
To stop receiving notification emails like this one, please contact
['"commits@whimsical.apache.org" <co...@whimsical.apache.org>'].