diff --git a/src/protocol/http/org/apache/jmeter/protocol/http/sampler/MeasuringConnectionManager.java b/src/protocol/http/org/apache/jmeter/protocol/http/sampler/MeasuringConnectionManager.java
index 4dc100674d..826897651c 100644
--- a/src/protocol/http/org/apache/jmeter/protocol/http/sampler/MeasuringConnectionManager.java
+++ b/src/protocol/http/org/apache/jmeter/protocol/http/sampler/MeasuringConnectionManager.java
@@ -106,9 +106,12 @@ public class MeasuringConnectionManager extends PoolingClientConnectionManager {
 
         @Override
         public void open(HttpRoute route, HttpContext context, HttpParams params) throws IOException {
-            handler.open(route, context, params);
-            if (sample != null) {
-                sample.connectEnd();
+            try {
+                handler.open(route, context, params);
+            } finally {
+                if (sample != null) {
+                    sample.connectEnd();
+                }
             }
         }
 
diff --git a/xdocs/changes.xml b/xdocs/changes.xml
index 431548c74a..a46936c979 100644
--- a/xdocs/changes.xml
+++ b/xdocs/changes.xml
@@ -240,6 +240,7 @@ Summary
     <li><bug>58209</bug>JMeter hang when testing javasampler because HashMap.put() is called from multiple threads without sync.</li>
     <li><bug>58301</bug>Use typed methods such as setInt, setDouble, setDate ... for prepared statement #27</li>
     <li><bug>58851</bug>Add a dependency to hamcrest-core to allow JUnit tests with annotations to work</li>
+    <li><bug>58947</bug>Connect metric is wrong when ConnectException occurs</li>
 </ul>
 
 <h3>Controllers</h3>