Merge pull request #453 from yma96/master

yma955 · web-flow · commit 848acaed85bd · 2025-03-18T15:37:43.000+08:00
Add timeout exception log for proxy usage
diff --git a/transports/httpclient/src/main/java/org/commonjava/maven/galley/transport/htcli/internal/AbstractHttpJob.java b/transports/httpclient/src/main/java/org/commonjava/maven/galley/transport/htcli/internal/AbstractHttpJob.java
@@ -147,6 +147,9 @@ else if ( !successStatuses.contains( sc ) )
                 catch ( final NoHttpResponseException | ConnectTimeoutException | SocketTimeoutException |
                               HttpHostConnectException e )
                 {
+                    logger.warn(
+                            "Timeout for url: {}, site: {}, could try to access with proxy, tries:{}, doProxy: {}, Reason: {}", url,
+                            site, tries, doProxy, e.getMessage() );
                     // For those are not in the iad2 tenant egress rules, will throw timeout so use the configured proxy to retry
                     if ( tries > 0 )
                     {

Original file line number	Diff line number	Diff line change
`@@ -147,6 +147,9 @@ else if ( !successStatuses.contains( sc ) )`
`147`	`147`	`catch ( final NoHttpResponseException \| ConnectTimeoutException \| SocketTimeoutException \|`
`148`	`148`	`HttpHostConnectException e )`
`149`	`149`	`{`
	`150`	`+ logger.warn(`
	`151`	`+ "Timeout for url: {}, site: {}, could try to access with proxy, tries:{}, doProxy: {}, Reason: {}", url,`
	`152`	`+ site, tries, doProxy, e.getMessage() );`
`150`	`153`	`// For those are not in the iad2 tenant egress rules, will throw timeout so use the configured proxy to retry`
`151`	`154`	`if ( tries > 0 )`
`152`	`155`	`{`