cloudfoundry-community / splunk-firehose-nozzle

Send CF component metrics, CF app logs, and CF app metrics to Splunk
Apache License 2.0
29 stars 29 forks source link

drained the response buffer to reuse the connection with HEC #304

Closed kashyap-splunk closed 2 years ago

kashyap-splunk commented 2 years ago

The HEC connections were not getting reused, and a new connection was getting created at every request. This created issues with the load-balancer in cloud deployment.

The connection logic was already implemented in a way to reuse the connection but the buffer was not drained for the successful requests, which prevented the connections to be reused. Fixed the issue.

luckyj5 commented 2 years ago

LGTM. Thanks @kashyap-splunk