Dealing with minor TODOs

Description

See #8.

• removed multiple TODOs as they were not necessary anymore
• rewrite TODO to note
• remove Port from remote/init KMS address to remove the need of splitting it later

goKMS/gnmiHandlers/kms/assignForwardingHandler.go

@@ -52,7 +52,6 @@ func (yh *AssignForwardingHandler) Update(c ygot.ValidatedGoStruct, jobs []*gnmi
 
 	var nextHopString string
 	var prevHopString string
-	var initiatingKmsAddressString string
 
 	if nextHop != nil {
 		nextHopString = nextHop.GetNodeId()
@@ -73,10 +72,10 @@ func (yh *AssignForwardingHandler) Update(c ygot.ValidatedGoStruct, jobs []*gnmi
 			log.Error("initiatingKmsAddress must have either an IP address or a hostname")
 			return fmt.Errorf("initiatingKmsAddress must have either an IP address or a hostname")
 		}
-		initiatingKmsAddressString = fmt.Sprintf("%s:%d", addressPrefix, initiatingKmsAddress.GetPort())
+
 		initKMS = &kms.RemoteKMS{
 			Id:      initiatingKmsAddress.GetNodeId(),
-			Address: initiatingKmsAddressString,
+			Address: addressPrefix,
 			Port:    initiatingKmsAddress.GetPort(),
 		}
 	}