ChugR commented on a change in pull request #472: Dispatch 1288 1 - policy for outbound connectors URL: https://github.com/apache/qpid-dispatch/pull/472#discussion_r267838761
########## File path: src/policy.c ########## @@ -1110,6 +1152,62 @@ void qd_policy_amqp_open(qd_connection_t *qd_conn) { } +void qd_policy_amqp_open_connector(qd_connection_t *qd_conn) { Review comment: The connector connection hasn't really opened yet. In this function policy is approving or denying the connection. Sure, the second Open has arrived but policy may slam the connection closed for some reason. The name is similar to the peer function that handles the remote open for incoming connections. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org For additional commands, e-mail: dev-h...@qpid.apache.org