Skip to content

Handle "connection reset by peer" - #982

Open
jammasterj89 wants to merge 1 commit into
ssllabs:masterfrom
jammasterj89:master
Open

jammasterj89 wants to merge 1 commit into
ssllabs:masterfrom
jammasterj89:master

Conversation

@jammasterj89

Copy link
Copy Markdown

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

goldeninetz commented Oct 31, 2024 •

Copy link
Copy Markdown

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