[ 
https://issues.apache.org/jira/browse/DIRKRB-231?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14518996#comment-14518996
 ] 

Kai Zheng commented on DIRKRB-231:
----------------------------------

Good work Yaning! Would you simplify the following two blocks? Thanks.

{code}
+        if (isTcp) {
+            KrbMessage krbResponse = kdcRequest.getReply();
+            int bodyLen = krbResponse.encodingLength();
+            ByteBuffer responseMessage = ByteBuffer.allocate(bodyLen + 4);
+            responseMessage.putInt(bodyLen);
+            krbResponse.encode(responseMessage);
+            responseMessage.flip();
+            return responseMessage;
+        } else {
+            KrbMessage krbResponse = kdcRequest.getReply();
+            int bodyLen = krbResponse.encodingLength();
+            ByteBuffer responseMessage = ByteBuffer.allocate(bodyLen);
+            krbResponse.encode(responseMessage);
+            responseMessage.flip();
+            return responseMessage;
+        }
{code}
{code}
+        if (!isTcp) {
+            ByteBuffer requestMessage = ByteBuffer.allocate(bodyLen);
+            kdcReq.encode(requestMessage);
+            requestMessage.flip();
+            try {
+                sendMessage(kdcRequest, requestMessage);
+            } catch (IOException e) {
+                throw new KrbException("sending message failed", e);
+            }
+        } else {
+            ByteBuffer requestMessage = ByteBuffer.allocate(bodyLen + 4);
+            requestMessage.putInt(bodyLen);
+            kdcReq.encode(requestMessage);
+            requestMessage.flip();
+            try {
+                sendMessage(kdcRequest, requestMessage);
+            } catch (IOException e) {
+                throw new KrbException("sending message failed", e);
+            }
{code}


> Support UDP transport for NettyKdcNetwork
> -----------------------------------------
>
>                 Key: DIRKRB-231
>                 URL: https://issues.apache.org/jira/browse/DIRKRB-231
>             Project: Directory Kerberos
>          Issue Type: New Feature
>            Reporter: Kai Zheng
>            Assignee: Xu Yaning
>         Attachments: DIRKRB-231-v1.patch
>
>
> This is TODO left to be done.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to