[CLIENT-3894] Setting user-agent allows a closed connection to return for tend

Author: khaf

CHANGELOG.md

@@ -1,23 +1,30 @@
 # Change History
 
+## November 14 2025: v8.4.2
+
+  **Fixes**
+    - [CLIENT_3894] Setting user-agent allows a closed connection to return for tend.
+
 ## November 4 2025: v8.4.1
-**Fixes**
-- [CLIENT-3821] Fixed issue where `panic: runtime error: invalid memory address` could occur after calling `usingTendConn(...)`.
+
+  **Fixes**
+    - [CLIENT-3821] Fixed issue where `panic: runtime error: invalid memory address` could occur after calling `usingTendConn(...)`.
 
 ## October 20 2025: v8.4.0
-**Improvements**
-- [CLIENT-3435] Update Go client to support FIPS.
-
-**Fixes**
-- [CLIENT-3821] Fixed issue where `Network Error from EOF` could occur after server migrations.
-- [CLIENT-3810] Fixed issue where nil pointer panic occurs on errToAerospikeErr.
-- [CLIENT-3796] Fixed issue where background queries for all records applied only partially.
-- [CLIENT-3744] Added server version check before sending the user-agent-set command.
-- [CLIENT-2418] Skipped orphan seeds without peers when other seeds have peers.
-- [CLIENT-3120] Replaced existing cluster node when a new peer shares the same node name but a different IP address.
-- [CLIENT-3546] Resolved inconsistencies between clients when reflecting configuration changes.
-- [CLIENT-3023] Fixed inability to repeat background queries using the same statement.
-- [CLIENT-3021] Ensured Statement.BinNames is empty for QueryExecute operations.
+
+  **Improvements**
+    - [CLIENT-3435] Update Go client to support FIPS.
+
+  **Fixes**
+    - [CLIENT-3821] Fixed issue where `Network Error from EOF` could occur after server migrations.
+    - [CLIENT-3810] Fixed issue where nil pointer panic occurs on errToAerospikeErr.
+    - [CLIENT-3796] Fixed issue where background queries for all records applied only partially.
+    - [CLIENT-3744] Added server version check before sending the user-agent-set command.
+    - [CLIENT-2418] Skipped orphan seeds without peers when other seeds have peers.
+    - [CLIENT-3120] Replaced existing cluster node when a new peer shares the same node name but a different IP address.
+    - [CLIENT-3546] Resolved inconsistencies between clients when reflecting configuration changes.
+    - [CLIENT-3023] Fixed inability to repeat background queries using the same statement.
+    - [CLIENT-3021] Ensured Statement.BinNames is empty for QueryExecute operations.
 
 ## August 29 2025: v8.3.0
 

go.mod

@@ -24,3 +24,7 @@ require (
 	golang.org/x/tools v0.31.0 // indirect
 	gopkg.in/check.v1 v1.0.0-20201130134442-10cb98267c6c // indirect
 )
+
+retract (
+	[v8.3.0, v8.4.1] // Problem with User Agent code returning a closed connection without an error.
+)

node.go

@@ -527,12 +527,11 @@ func (nd *Node) newTendConnection() (*Connection, Error) {
 		return nil, err
 	}
 
-	serverMinVersion, _ := version.Parse("8.1.0.0")
+	serverMinVersion := &version.Version{Major: 8, Minor: 1, Patch: 0, Build: 0}
 	if nd.version.IsGreaterOrEqual(serverMinVersion) {
 		if err := nd.sendUserAgentId(conn); err != nil {
-			// If setting user agent failed, we still return the connection
-			// as it is already authenticated and usable.
 			logger.Logger.Warn("Error setting user agent for node %s: %s", nd.String(), err.Error())
+			return nil, err
 		}
 	}