BUG-2475: Fix keepalives not being sent 80/14380/2
authorRobert Varga <rovarga@cisco.com>
Thu, 22 Jan 2015 14:17:38 +0000 (15:17 +0100)
committerRobert Varga <rovarga@cisco.com>
Thu, 22 Jan 2015 14:23:33 +0000 (15:23 +0100)
As it turns out, we failed to update channel configuration when MD5 keys
were not specified.

Change-Id: I5c2a5983c0797d032bb058974aac82254b554041
Signed-off-by: Robert Varga <rovarga@cisco.com>
bgp/rib-impl/src/main/java/org/opendaylight/protocol/bgp/rib/impl/BGPDispatcherImpl.java

index 38abcf2637545fd911ae8b1a5b9daf7f3b48d413..9ac117939ceb4acabc860a4107685636647fb6e1 100644 (file)
@@ -128,9 +128,10 @@ public final class BGPDispatcherImpl extends AbstractDispatcher<BGPSessionImpl,
             }
             b.channelFactory(this.cf);
             b.option(MD5ChannelOption.TCP_MD5SIG, this.keys);
-            // Make sure we are doing round-robin processing
-            b.option(ChannelOption.MAX_MESSAGES_PER_READ, 1);
         }
+
+        // Make sure we are doing round-robin processing
+        b.option(ChannelOption.MAX_MESSAGES_PER_READ, 1);
     }
 
     @Override
@@ -141,9 +142,10 @@ public final class BGPDispatcherImpl extends AbstractDispatcher<BGPSessionImpl,
             }
             b.channelFactory(this.scf);
             b.option(MD5ChannelOption.TCP_MD5SIG, this.keys);
-            // Make sure we are doing round-robin processing
-            b.childOption(ChannelOption.MAX_MESSAGES_PER_READ, 1);
         }
+
+        // Make sure we are doing round-robin processing
+        b.childOption(ChannelOption.MAX_MESSAGES_PER_READ, 1);
     }
 
 }