Merge pull request #224 from rolandshoemaker/182-return

Return 500 for multiple certs with same short serial
2015-05-26 15:53:02 -07:00 · 2015-05-26 15:53:02 -07:00 · 5d82c5f994
parent 6c6199023d 5541a8ab37
commit 5d82c5f994
1 changed files with 5 additions and 1 deletions
--- a/wfe/web-front-end.go
+++ b/wfe/web-front-end.go
@ -709,7 +709,11 @@ func (wfe *WebFrontEndImpl) Certificate(response http.ResponseWriter, request *h

 		cert, err := wfe.SA.GetCertificateByShortSerial(serial)
 		if err != nil {
-			wfe.sendError(response, "Not found", err, http.StatusNotFound)
+			if strings.HasPrefix(err.Error(), "gorp: multiple rows returned") {
+				wfe.sendError(response, "Multiple certificates with same short serial", err, http.StatusInternalServerError)
+			} else {
+				wfe.sendError(response, "Not found", err, http.StatusNotFound)
+			}
 			return
 		}