Github user ted-ross commented on a diff in the pull request:

    https://github.com/apache/qpid-dispatch/pull/326#discussion_r198583346
  
    --- Diff: src/server.c ---
    @@ -651,8 +651,9 @@ static void on_connection_bound(qd_server_t *server, 
pn_event_t *e) {
             pn_sasl_set_allow_insecure_mechs(sasl, 
config->allowInsecureAuthentication);
             sys_mutex_unlock(ctx->server->lock);
     
    -        qd_log(ctx->server->log_source, QD_LOG_INFO, "Accepted connection 
to %s from %s",
    -               name, ctx->rhost_port);
    +        qd_log(ctx->server->log_source, QD_LOG_INFO,
    --- End diff --
    
    There appear to be more arguments than format directives in this call.
    Why did you remove the information about the remote host?


---

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org

Reply via email to