Skip to content

Commit 31fd393

Browse files
committed
fix:http health checker timing bug.
1 parent eb00a87 commit 31fd393

File tree

2 files changed

+4
-3
lines changed
  • polaris-plugins/polaris-plugins-healthchecker/healthchecker-http/src/main/java/com/tencent/polaris/plugins/outlier/detector/http

2 files changed

+4
-3
lines changed

polaris-plugins/polaris-plugins-healthchecker/healthchecker-http/src/main/java/com/tencent/polaris/plugins/outlier/detector/http/HttpHealthChecker.java

+3-2
Original file line numberDiff line numberDiff line change
@@ -36,11 +36,12 @@
3636
import com.tencent.polaris.specification.api.v1.fault.tolerance.FaultDetectorProto.FaultDetectRule.Protocol;
3737
import com.tencent.polaris.specification.api.v1.fault.tolerance.FaultDetectorProto.HttpProtocolConfig;
3838
import com.tencent.polaris.specification.api.v1.fault.tolerance.FaultDetectorProto.HttpProtocolConfig.MessageHeader;
39+
import org.slf4j.Logger;
40+
3941
import java.io.IOException;
4042
import java.io.OutputStream;
4143
import java.nio.charset.StandardCharsets;
4244
import java.util.List;
43-
import org.slf4j.Logger;
4445

4546

4647
/**
@@ -102,8 +103,8 @@ public DetectResult detectInstance(Instance instance, FaultDetectRule faultDetec
102103
byte[] input = body.getBytes(StandardCharsets.UTF_8);
103104
outputStream.write(input, 0, input.length);
104105
}
105-
long delayMillis = System.currentTimeMillis() - startTimeMillis;
106106
int responseCode = conn.getResponseCode();
107+
long delayMillis = System.currentTimeMillis() - startTimeMillis;
107108

108109
RetStatus retStatus = responseCode >= 200 && responseCode < 500 ? RetStatus.RetSuccess : RetStatus.RetFail;
109110
return new DetectResult(responseCode, delayMillis, retStatus);

pom.xml

+1-1
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,7 @@
6464

6565
<properties>
6666
<!-- Project revision -->
67-
<revision>1.12.11</revision>
67+
<revision>1.12.12</revision>
6868

6969
<timestamp>${maven.build.timestamp}</timestamp>
7070
<skip.maven.deploy>false</skip.maven.deploy>

0 commit comments

Comments
 (0)