log an error to sentry if an upstream service responds with `429 Too …

…Many Requests` (#9505)

* log an error to sentry if an upstream service responds with 429 Too Many Requests

* fix tests

core/base-service/check-error-response.js

@@ -1,3 +1,4 @@
+import log from '../server/log.js'
 import { NotFound, InvalidResponse, Inaccessible } from './errors.js'
 
 const defaultErrorMessages = {
@@ -25,6 +26,13 @@ export default function checkErrorResponse(httpErrors = {}) {
         error = new InvalidResponse(props)
       }
     }
+
+    if (res.statusCode === 429) {
+      log.error(
+        new Error(`429 Too Many Requests calling ${res.requestUrl.origin}`),
+      )
+    }
+
     if (error) {
       error.response = res
       error.buffer = buffer

core/base-service/check-error-response.spec.js

@@ -47,7 +47,7 @@ describe('async error handler', function () {
 
   context('when status is 429', function () {
     const buffer = Buffer.from('some stuff')
-    const res = { statusCode: 429 }
+    const res = { statusCode: 429, requestUrl: new URL('https://example.com/') }
 
     it('throws InvalidResponse', async function () {
       try {