There is a problem with the retry logic, which may lead to an infinite loop

[hxy1991]

Reporting bugs

If the execution of the request takes longer than "readTimeoutMs" and the request always fails, the (System.currentTimeMillis() > endTime) condition is always true, resulting in an infinite loop.

ServerHttpAgent#httpGet

// ServerHttpAgent.httpGet

public HttpResult httpGet(String path, List<String> headers, List<String> paramValues, String encoding,
	long readTimeoutMs) throws IOException {

	final long endTime = System.currentTimeMillis() + readTimeoutMs;

	……
	
	do {
		
		……

		HttpResult result = HttpSimpleClient.httpGet(
				getUrl(serverListMgr.getCurrentServerAddr(), path, isSSL), newHeaders, paramValues, encoding,
				readTimeoutMs, isSSL);
		if (result.code == HttpURLConnection.HTTP_INTERNAL_ERROR
				|| result.code == HttpURLConnection.HTTP_BAD_GATEWAY
				|| result.code == HttpURLConnection.HTTP_UNAVAILABLE) {
			log.error("NACOS ConnectException", "currentServerAddr:{}. httpCode:",
					new Object[] { serverListMgr.getCurrentServerAddr(), result.code });
		} else {
			return result;
		}

	} while (System.currentTimeMillis() > endTime);

	……
	
}

See Also:

• ServerHttpAgent#httpPost

• ServerHttpAgent#httpDelete