TLS support

[openflowjava.git] / openflow-protocol-impl / src / main / java / org / opendaylight / openflowjava / protocol / impl / core / OFFrameDecoder.java

diff --git a/openflow-protocol-impl/src/main/java/org/opendaylight/openflowjava/protocol/impl/core/OFFrameDecoder.java b/openflow-protocol-impl/src/main/java/org/opendaylight/openflowjava/protocol/impl/core/OFFrameDecoder.java
index dab263a2da458df32ce07fea8c0ff6af6e5fc188..d074e93e5a70f0f2d6c1f40c8a4d4273d344a487 100644 (file)
--- a/openflow-protocol-impl/src/main/java/org/opendaylight/openflowjava/protocol/impl/core/OFFrameDecoder.java
+++ b/openflow-protocol-impl/src/main/java/org/opendaylight/openflowjava/protocol/impl/core/OFFrameDecoder.java
@@ -15,6 +15,7 @@ import io.netty.handler.codec.ByteToMessageDecoder;
 
 import java.util.List;
 
+import org.opendaylight.openflowjava.protocol.impl.connection.ConnectionFacade;
 import org.opendaylight.openflowjava.protocol.impl.util.ByteBufUtils;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
@@ -29,12 +30,16 @@ public class OFFrameDecoder extends ByteToMessageDecoder {
     public static final byte LENGTH_OF_HEADER = 8;
     private static final byte LENGTH_INDEX_IN_HEADER = 2;
     private static final Logger LOGGER = LoggerFactory.getLogger(OFFrameDecoder.class);
+    private ConnectionFacade connectionFacade;
+    private boolean started = false;
 
     /**
      * Constructor of class.
+     * @param connectionFacade 
      */
-    public OFFrameDecoder() {
+    public OFFrameDecoder(ConnectionFacade connectionFacade) {
         LOGGER.trace("Creating OFFrameDecoder");
+        this.connectionFacade = connectionFacade;
     }
 
     @Override
@@ -45,9 +50,14 @@ public class OFFrameDecoder extends ByteToMessageDecoder {
 
     @Override
     protected void decode(ChannelHandlerContext chc, ByteBuf bb, List<Object> list) throws Exception {
+        if (!started) {
+            connectionFacade.fireConnectionReadyNotification();
+            started = true;
+        }
         int readableBytes = bb.readableBytes();
         if (readableBytes < LENGTH_OF_HEADER) {
-            LOGGER.debug("skipping bb - too few data for header: " + readableBytes);
+            LOGGER.debug("skipping bytebuf - too few bytes for header: " + readableBytes + " < " + LENGTH_OF_HEADER );
+            LOGGER.debug("bb: " + ByteBufUtils.byteBufToHexString(bb));
             return;
         }
         
@@ -55,11 +65,9 @@ public class OFFrameDecoder extends ByteToMessageDecoder {
         LOGGER.debug("length of actual message: {}", length);
         
         if (readableBytes < length) {
-            if (LOGGER.isDebugEnabled()) {
-                LOGGER.debug("skipping bb - too few data for msg: " +
+                LOGGER.debug("skipping bytebuf - too few bytes for msg: " +
                         readableBytes + " < " + length);
-                LOGGER.debug("bb: " + ByteBufUtils.byteBufToHexString(bb));
-            }
+                LOGGER.debug("bytebuffer: " + ByteBufUtils.byteBufToHexString(bb));
             return;
         }
         LOGGER.debug("OF Protocol message received, type:{}", bb.getByte(bb.readerIndex() + 1));