Merge pull request #1333 from cisagov/za/fix-multiple-pool-objects

EPP Connection Pool fix

src/epplibwrapper/socket.py

@@ -78,7 +78,7 @@ def test_connection_success(self):
                 self.client.connect()
                 response = self.client.send(self.login)
             except LoginError as err:
-                if err.should_retry() and counter < 3:
+                if err.should_retry() and counter < 10:
                     counter += 1
                     sleep((counter * 50) / 1000)  # sleep 50 ms to 150 ms
                 else:  # don't try again

src/epplibwrapper/utility/pool.py

@@ -97,7 +97,7 @@ def get_connections(self):
     def kill_all_connections(self):
         """Kills all active connections in the pool."""
         try:
-            if len(self.conn) > 0:
+            if len(self.conn) > 0 or len(self.greenlets) > 0:
                 gevent.killall(self.greenlets)
 
                 self.greenlets.clear()
@@ -116,7 +116,7 @@ def populate_all_connections(self):
         If any connections exist, kill them first.
         Based off of the __init__ definition for geventconnpool.
         """
-        if len(self.conn) > 0:
+        if len(self.conn) > 0 or len(self.greenlets) > 0:
             self.kill_all_connections()
 
         # Setup the lock