@henningw commented on this pull request.

Thanks for the PR. I only noticed on small spelling error in the log. Regarding the format check failure, no idea why this happens. Lets wait for eventual other comments and then it can be merged. I can fix eventual remaining errors in the formatting by manually executing the format tool.


In src/modules/rabbitmq/rabbitmq.c:

> @@ -557,25 +561,52 @@ static int rabbitmq_connect(amqp_connection_state_t *conn)
 	int ret;
 	int log_ret;
 	//	amqp_rpc_reply_t reply;
+	
+	// amqp_ssl_init_called should only be called once
+	if(amqp_info.ssl && !amqp_ssl_init_called) {
+		amqp_set_initialize_ssl_library(1);
+		amqp_ssl_init_called = 1;
+		LM_DBG("AMQP SSL library inilialized\n");

Spelling


Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: <kamailio/kamailio/pull/3511/review/1529168735@github.com>