Skip to content

Conversation

jammasterj89
Copy link

I was seeing a lot of 'connection reset by peer' errors, so this change helps prevent the test from crashing when they occur.

add error handling for connection reset by peer
@goldeninetz
Copy link

goldeninetz commented Oct 31, 2024

There is a retry count already in the code, could we leverage that as well?
master...goldeninetz:ssllabs-scan:sa2477errorhandling

				log.Fatalf("[ERROR] HTTP request failed: %v (ref#2)", err.Error())
				if retryCount < 5 {
//					if logLevel >= LOG_WARNING {
						log.Printf("[WARNING] Attempt %v HTTP request failed: %v (ref#2)", retryCount, err.Error())
//					}
				} else {
					log.Fatalf("[ERROR] HTTP request failed: %v (ref#2)", err.Error())
				}
			}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants