WFE: Only log warnings for non-limit errors (letsencrypt#7717)

Only log errors from `wfe.checkNewOrderLimits` and `wfe.checkNewAccountLimits` if they're not `errors.RateLimit`.
mcpherrinm · Sep 26, 2024 · 3451952 · 3451952
1 parent 8c009f2
commit 3451952
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/wfe2/wfe.go b/wfe2/wfe.go
@@ -801,8 +801,9 @@ func (wfe *WebFrontEndImpl) NewAccount(
 				wfe.sendError(response, logEvent, probs.RateLimited(err.Error()), err)
 				return
 			}
+		} else {
+			wfe.log.Warning(err.Error())
 		}
-		wfe.log.Warning(err.Error())
 	}
 
 	var newRegistrationSuccessful bool
@@ -2383,8 +2384,9 @@ func (wfe *WebFrontEndImpl) NewOrder(
 				wfe.sendError(response, logEvent, probs.RateLimited(err.Error()), err)
 				return
 			}
+		} else {
+			wfe.log.Warning(err.Error())
 		}
-		wfe.log.Warning(err.Error())
 	}
 
 	var newOrderSuccessful bool