Hello, as far as I know axios does not work on edge function right? but why is this api endpoint of

Hello, as far as I know axios does not work on edge function right? but why is this api endpoint of mine working just fine

James•9/29/24, 3:37 PM

magic

Kkoribot Hello, as far as I know axios does not work on edge function right? but why is t...

yuripave•9/30/24, 1:24 AM

https://github.com/axios/axios/pull/6371

GitHub

feat(adapter): add fetch adapter; by DigitalBrainJS · Pull Request ...

The goal of this PR is to add a minimal fetch implementation that provides the same capabilities as the xhr adapter without major changes, allowing it to be released in Axios 1.x.
To avoid breaking...

chof64•9/30/24, 3:39 PM

Hello, trying to learn and try deploying an existing nextjs project to pages, I'm following the ssr guide. But I got a question.

In the docs, a library @cloudflare/next-on-pages@cloudflare/next-on-pages is needed, and when I installed it as dev dependency. I now get 11 vulnerabilities in my project. But before installing the dev dependency these vulnerabilities don't exist. Is there a way to fix these or maybe a resource that talks about these so that I can do further research?

tadhglewis•9/30/24, 4:20 PM

https://snyk.io/advisor/npm-package/@cloudflare/next-on-pages

Snyk Advisor

View @cloudflare/next-on-pa… on Snyk Open Source Advisor

Find the best open-source package for your project with Snyk Open Source Advisor. Explore over 1 million open source packages.

tadhglewis•9/30/24, 4:21 PM

npm audits typically show a lot of stuff due to how many dependencies npm packages have (you'd need to double check the source of the 11 vulerabilities)... Majority of it (esp for dev deps) is not applicable. Most of the time you just need to re-generate the lock file which updates downstream dependencies

Puliczek•10/1/24, 6:43 PM

Are Next.js Workers more expensive than Next.js Pages? Does one HTTP request equal one worker request?

James•10/1/24, 6:50 PM

it's the same

James•10/1/24, 6:50 PM

Pages uses workers

Sevrene•10/1/24, 8:37 PM

Attempting to embed Sanity Studio into a Cloudflare Pages deploy and since it must run on edge it falls into the worker size limit
I am assuming my solution is to just upgrade to a paid plan to fit within the larger size limit and I'm not configuring something incorrectly here?

James•10/1/24, 9:50 PM

correct

JJames it's the same

tadhglewis•10/1/24, 11:01 PM

That's not necessarily true, it's one request but can be more expensive. @Puliczek I replied to you on Open NextJS discord

tadhglewis•10/1/24, 11:01 PM

Tldr: higher CPU and page load but limited testing

Puliczek•10/2/24, 6:32 AM

Thank you so much for your analysis. I will wait longer before migrating. Also, there is no proper tutorial on how to migrate from Pages to Workers.

面

面条•10/2/24, 3:27 PM

Middleware ？

James•10/2/24, 4:41 PM

Middleware works

AA•10/2/24, 10:33 PM

I keep having this problem, when I update next-on-pages, my project stops working with a "XMLHttpRequest is not defined" any idea how to troubleshoot this?

AA•10/2/24, 10:33 PM

I think i've mentioned it previously

AA•10/2/24, 10:40 PM

yea soon as i force 1.9.0 everything works ok

AAA any differences between 1.9.0 and 1.13.2 that would give me an "ReferenceError: ...

AA•10/3/24, 12:28 AM

yea, i knew i'd asked about it previously

James•10/3/24, 7:09 AM

That doesnt sound related to next-on-pages

bun•10/3/24, 11:21 AM

anyone had success integrating sentry.io on next-on-pages ? or is it not working? @Baran

AAA I keep having this problem, when I update next-on-pages, my project stops workin...

bun•10/3/24, 12:18 PM

seems like you are using axios on edge runtime

bun•10/3/24, 12:19 PM

maybe not use axios ? and use requestlyrequestly or some modern fetch library

JJames That doesnt sound related to next-on-pages

AA•10/3/24, 12:53 PM

I agree, which is why i'm confused. but the only thing I'm changing is the version of next-on-pages

AA•10/3/24, 12:53 PM

it makes me wonder if there is a dependency interaction that i'm not aware of

AA•10/3/24, 1:01 PM

the actual code change that made this work.. Tho the package.json change is massive

AA•10/3/24, 1:02 PM

removing this line, or changing the version starts producing the XMLHttpRequest

AA•10/3/24, 1:06 PM

yea. line change with that add on is +10354 -3237

AAA the actual code change that made this work.. Tho the package.json change is mas...

bun•10/3/24, 1:13 PM

are you using axios or not?

Bbun are you using axios or not?

AA•10/3/24, 1:14 PM

not directly, and i dont see it in any sub dependencies

AA•10/3/24, 1:15 PM

most everything is through a graphql library that could use axios, but has an override to use fetch

AA•10/3/24, 1:16 PM

axios does not appear in the package.lock

bun•10/3/24, 1:17 PM

what about cross-fetch ?

AA•10/3/24, 1:17 PM

indeed it does

bun•10/3/24, 1:18 PM

yeah that could be the cause, is there a way you can initiate a graphql client using your own fetch?

bun•10/3/24, 1:18 PM

it should have options

AA•10/3/24, 1:18 PM

yea you can override the fetch in the graphql client. if i dont do that it doesnt work on any version of next-on-pages

AA•10/3/24, 1:19 PM

required by node_modules/fbjs "node_modules/graphql-request":

AA•10/3/24, 1:24 PM

not familiar with fbjs, hmm

AA•10/3/24, 1:30 PM

not sure this explains why changing the next-on-pages version breaks this either

James•10/3/24, 1:55 PM

maybe it's updating other dependencies that aren't locked to specific patches too

Bbun anyone had success integrating sentry.io on next-on-pages ? or is it not working...

Baran•10/3/24, 3:42 PM

There is a fix coming to next-on-pages, so sentry will be implemented soon. I dont know when the next release date is

JJames maybe it's updating other dependencies that aren't locked to specific patches to...

AA•10/3/24, 4:25 PM

yea i'm wondering about that. its weird because if i changee this to the 1.10 or later it breaks, and then back it works.. just need to trace every dep i guess