wfe: on rate limit error, serve 500

This affects NewAccount and NewOrder.

web/context.go

@@ -35,12 +35,7 @@ type RequestEvent struct {
 	Slug           string   `json:",omitempty"`
 	InternalErrors []string `json:",omitempty"`
 	Error          string   `json:",omitempty"`
-	// If there is an error checking the data store for our rate limits
-	// we ignore it, but attach the error to the log event for analysis.
-	// TODO(#7796): Treat errors from the rate limit system as normal
-	// errors and put them into InternalErrors.
-	IgnoredRateLimitError string `json:",omitempty"`
-	UserAgent             string `json:"ua,omitempty"`
+	UserAgent      string   `json:"ua,omitempty"`
 	// Origin is sent by the browser from XHR-based clients.
 	Origin string                 `json:",omitempty"`
 	Extra  map[string]interface{} `json:",omitempty"`

wfe2/wfe.go

@@ -798,7 +798,8 @@ func (wfe *WebFrontEndImpl) NewAccount(
 			wfe.sendError(response, logEvent, probs.RateLimited(err.Error()), err)
 			return
 		} else {
-			logEvent.IgnoredRateLimitError = err.Error()
+			wfe.sendError(response, logEvent, web.ProblemDetailsForError(err, "While checking rate limits"), err)
+			return
 		}
 	}
 
@@ -2408,7 +2409,8 @@ func (wfe *WebFrontEndImpl) NewOrder(
 				return
 			}
 		} else {
-			logEvent.IgnoredRateLimitError = err.Error()
+			wfe.sendError(response, logEvent, web.ProblemDetailsForError(err, "While checking rate limits"), err)
+			return
 		}
 	}