TypeError: Cannot read properties of undefined (reading 'status')

jvidalv commented 1 year ago

Describe the bug

For some unknown reason urql is erroring in this file, and line, sometimes:

@urql+core@4.0.2_graphql@16.6.0/node_modules/@urql/core/dist/urql-core-chunk.mjs

yield makeErrorResult(r, (s.status < 200 || s.status >= 300) && s.statusText ? new Error(s.statusText) : e, s);

Do we have any idea that what could be the cuplrit?

Reproduction

https://grafbase.com

Urql version

urql 4.04

Validations

[X] I can confirm that this is a bug report, and not a feature request, RFC, question, or discussion, for which GitHub Discussions should be used
[X] Read the docs.
[X] Follow our Code of Conduct

kitten commented 1 year ago

Just to point this out, the version of urql (the React bindings) isn't relevant here, but, by chance, I can see in your file path that you're on @urql/core@4.0.2.

There's already been a patch for this regression, so if you upgrade it'll be fixed ✌️ https://github.com/urql-graphql/urql/pull/3131

kitten commented 1 year ago

Hiya again :wave: Completely unrelated, but I just took a look at your sourcemaps, since I believe the Grafbase dashboard is what you're working on and I noticed your sourcemaps seem to be publicly accessible ✌️

I'll make it brief, but since it's interesting to me to see how people use urql, since we don't really get a close look that often I did take a quick peek, and I think there's some easy wins here:

@0no-co/graphql.web is out-of-date. There's been a couple of small patches. I think you're on 1.0.0 and you should be able to upgrade to @0no-co/graphql.web@1.0.4 safely. This just fixes a couple of small eager matches
wonka seems to be duplicated (you've got 6.3.1 and 6.3.2). This won't cause issues but if you pnpm dedupe it'll hopefully just give you a couple of bytes back for free.
Now, completely unrelated, I can see two versions of graphql in your bundle, but I obviously didn't go as far as looking why they're in there and what depends on them.
- Not sure if it's only packages/ui/src/utils/graphql/* that depends on it, but basically if it's only parsing you need, you could give graphql-web-lite a try if you're interested 👀 That's obviously a bit more work than just the upgrades above, but thought I'd mention it

Hope this helps! Sorry, I was basically just curious 😄

jvidalv commented 1 year ago

@0no-co/graphql.web

Hi Kitten! Thank you very much.

We have done what you mention, upgraded to the latest version of urql and next-urql, but we are still seing this status error (sometimes after a 401).

I don't know if you can point me to any other direction.

Thanks!

urql-graphql / urql