Skip to content

Handle "connection reset by peer"#982

Open
jammasterj89 wants to merge 1 commit intossllabs:masterfrom
jammasterj89:master
Open

Handle "connection reset by peer"#982
jammasterj89 wants to merge 1 commit intossllabs: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
Copy link
Copy Markdown

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