Return 429 Too Many Requests for gRPC error code ResourceExhausted …

…from Trillian (#1401) * Return 429 Too Many Requests for gRPC error code `ResourceExhausted` * Update CHANGELOG.md
google · Apr 15, 2024 · 2aea376 · 2aea376
1 parent 1ce7287
commit 2aea376
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -6,6 +6,10 @@
 
 * Adds the ability for a CT client to disable root compatibile checking: https://github.com/google/certificate-transparency-go/pull/1258
 
+### Fixes
+
+* Return 429 Too Many Requests for gRPC error code `ResourceExhausted` from Trillian.
+
 ## v1.1.8
 
 * Recommended Go version for development: 1.21

diff --git a/trillian/ctfe/handlers.go b/trillian/ctfe/handlers.go
@@ -1118,8 +1118,10 @@ func (li *logInfo) toHTTPStatus(err error) int {
 		return http.StatusBadRequest
 	case codes.NotFound:
 		return http.StatusNotFound
-	case codes.PermissionDenied, codes.ResourceExhausted:
+	case codes.PermissionDenied:
 		return http.StatusForbidden
+	case codes.ResourceExhausted:
+		return http.StatusTooManyRequests
 	case codes.Unauthenticated:
 		return http.StatusUnauthorized
 	case codes.FailedPrecondition: