feat: access-api createStoreProxy uses patched fetch that will log an…

…y non-ok responses (to debug) (#364)

…that logs unexpected responses with status=530

Motivation:
* gather more info for [this error happening in
production](#363 (comment))

Plan
* gather info, then revert this PR

packages/access-api/src/service/index.js

@@ -21,7 +21,28 @@ import * as uploadApi from './upload-api-proxy.js'
  */
 export function service(ctx) {
   return {
-    store: uploadApi.createStoreProxy(ctx),
+    store: uploadApi.createStoreProxy({
+      ...ctx,
+      fetch: patchedFetch(
+        globalThis.fetch.bind(globalThis),
+        async (response, fetchArgs) => {
+          if (!response.ok) {
+            try {
+              ctx.log.warn(
+                `unexpected non-ok response from fetch in createStoreProxy`,
+                await describeFetch(response, fetchArgs)
+              )
+            } catch (error) {
+              ctx.log.error(
+                new Error(`error describing/logging fetch response`, {
+                  cause: error,
+                })
+              )
+            }
+          }
+        }
+      ),
+    }),
     upload: uploadApi.createUploadProxy(ctx),
 
     voucher: {
@@ -151,3 +172,46 @@ export function service(ctx) {
     },
   }
 }
+
+/**
+ * Wrap `fetch` producing a new fetch that will pass any responses it fetches to a cb.
+ *
+ * @param {typeof globalThis.fetch} fetch
+ * @param {(response: Response, fetchArgs: Parameters<typeof globalThis.fetch>) => Promise<void>} onResponse - handles each response
+ * @returns {typeof globalThis.fetch}
+ */
+function patchedFetch(fetch, onResponse) {
+  /** @type {typeof globalThis.fetch} */
+  const fetchWithLog = async (input, init) => {
+    const response = await fetch(input, init)
+    await onResponse(response, [
+      input instanceof URL ? input.toString() : input,
+      init,
+    ])
+    return response
+  }
+  return fetchWithLog
+}
+
+/**
+ * Given a fetch invocation and the response it produced,
+ * return an object that can be logged to describe the request/response fully.
+ *
+ * @param {Response} response
+ * @param {Parameters<typeof globalThis.fetch>} fetchArgs
+ */
+async function describeFetch(response, fetchArgs) {
+  return {
+    request: fetchArgs,
+    response: {
+      type: response.type,
+      ok: response.ok,
+      redirected: response.redirected,
+      headers: [...response.headers],
+      status: response.status,
+      statusText: response.statusText,
+      url: response.url,
+      text: await response.clone().text(),
+    },
+  }
+}