fetch doesn’t handle try/catch when http2 request fails

any ideas how to fix this in pooledMap? I wrapped my entire fetch request with Try catch and it still crashes

TypeError: error sending request for url (https://briskreader.com/api/1/fetch?url=https://www.scmp.com/news/hong-kong/law-and-crime/article/3157622/hong-kong-emergency-services-search-hiker-missing?utm_source=rss_feed): http2 error: connection error received: not a result of an error
    at async mainFetch (deno:ext/fetch/26_fetch.js:266:14)
error: Uncaught (in promise) AggregateError: Threw while mapping.
        new AggregateError(errors, "Threw while mapping."),
        ^
    at https://deno.land/std@0.115.1/async/pool.ts:63:9

this happens if my api goes down while I’m trying to send a request to it…if nginx doesnj’t server a 200 then my script crashes

i’m using systemd and if i crash the api it restarts, but during that time my pooledMap crashes from my script.

i basically did try { ... fetch() } catch(err) { return Promise.resolve() }

doesnt’ seem to work with pooledMap

[ralyodio]

So I guess my fetch pattern sucked and doesn’t do what I thought it did.

I replaced:

cons res = await fetch(url).catch((err) => {
     console.error(err);
     return Promise.resolve()
});

with a real try/catch:

try {
  cons res = await fetch(url);
  if (res.ok) {
    body = await res.text();
 }
} catch(err) {
   console.error(err);
return Promise.resolve();
}

and now it doesn’t crash.

[ralyodio]

I also get this error sometimes:

TypeError: Fetch failed: Maximum number of redirects (20) reached

TypeError: Fetch failed: Maximum number of redirects (20) reached
    at deno:ext/fetch/26_fetch.js:440:27
error: Uncaught (in promise) AggregateError: Threw while mapping.
        new AggregateError(errors, "Threw while mapping."),
        ^
    at https://deno.land/std@0.116.0/async/pool.ts:63:9

try/catch is ignored