out_slack: destroy http client before releasing conn

edsiper · edsiper · commit 866f89a16a61 · 2026-04-09T15:24:17.000-06:00
Signed-off-by: Eduardo Silva &lt;eduardo@chronosphere.io&gt;
diff --git a/plugins/out_slack/slack.c b/plugins/out_slack/slack.c
@@ -282,8 +282,8 @@ static void cb_slack_flush(struct flb_event_chunk *event_chunk,
         out_ret = FLB_RETRY;
     }
 
-    flb_upstream_conn_release(u_conn);
     flb_http_client_destroy(c);
+    flb_upstream_conn_release(u_conn);
     flb_sds_destroy(out_buf);
     FLB_OUTPUT_RETURN(out_ret);
 }

Original file line number	Diff line number	Diff line change
`@@ -282,8 +282,8 @@ static void cb_slack_flush(struct flb_event_chunk *event_chunk,`
`282`	`282`	`out_ret = FLB_RETRY;`
`283`	`283`	`}`
`284`	`284`
`285`		`- flb_upstream_conn_release(u_conn);`
`286`	`285`	`flb_http_client_destroy(c);`
	`286`	`+ flb_upstream_conn_release(u_conn);`
`287`	`287`	`flb_sds_destroy(out_buf);`
`288`	`288`	`FLB_OUTPUT_RETURN(out_ret);`
`289`	`289`	`}`