network: explicitly close connection for log event (#469)

xinlili-statsig · web-flow · commit 93cc104b1de8 · 2025-10-29T10:38:04.000-07:00
Context: One of the company -- affirm reported that they are seeing higher econnreset error after we started using session and cause higher log event drop rate. I did some research this might be because when use request.session() connection headers is set to be keep-alive I am testing more https://statsigconnect.slack.com/archives/C0667QXTWCU/p1761587654203119?thread_ts=1761587241.636689&cid=C0667QXTWCU Tested and verified request.session() sends extra header on connection:keep-alive And my current approach works
diff --git a/statsig/http_worker.py b/statsig/http_worker.py
@@ -154,6 +154,7 @@ def log_events(
         disable_compression = _SDK_Configs.on("stop_log_event_compression")
         additional_headers = {
             "STATSIG-RETRY": str(retry),
+            "Connection": "close"
         }
         if headers is not None:
             additional_headers.update(headers)

Original file line number	Diff line number	Diff line change
`@@ -154,6 +154,7 @@ def log_events(`
`154`	`154`	`disable_compression = _SDK_Configs.on("stop_log_event_compression")`
`155`	`155`	`additional_headers = {`
`156`	`156`	`"STATSIG-RETRY": str(retry),`
	`157`	`+ "Connection": "close"`
`157`	`158`	`}`
`158`	`159`	`if headers is not None:`
`159`	`160`	`additional_headers.update(headers)`