it's the same

JamesOP•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?

JamesOP•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 ？

JamesOP•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

JamesOP•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

JamesOP•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

BBaran There is a fix coming to next-on-pages, so sentry will be implemented soon. I do...

bun•10/3/24, 4:59 PM

Nice

bun•10/3/24, 4:59 PM

The only jssue with sentry and nextjs is the bundle size? Or is there something else @Baran

mr nooli•10/3/24, 5:00 PM

If bundle size is the issue, I think take a look at posthog - it's quite nice pricing wise and I think offers a somewhat similar service

AA•10/3/24, 8:29 PM

i wonder if there is a way to tell it to ignore the cross-fetch library.. it may be stealing the 'fetch' word, and not doing a good job on cloudflare

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

just tried the 4.0 version of that lib which claims cf support, with no go

Mmr nooli If bundle size is the issue, I think take a look at posthog - it's quite nice pr...

bun•10/4/24, 1:43 AM

does it work on next-on-pages?

bun•10/4/24, 2:56 PM

@mr nooli

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

Huh, if cross-fetch is an included package in my project, 'fetch' gets overwritten globally somehow if I'm using a specific combination of packages.. regardless if I import cross-fetch in my specific file.

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

feels like a build problem, tho, is there a reference somewhere to the original fetch i can use?

AA•10/4/24, 5:00 PM

tho i wonder if i can do some next.config.js magic and tell it to use a local file for cross-fetch package

AA•10/4/24, 5:37 PM

heh, asked chatgpt.. https://chatgpt.com/share/6700278e-bc4c-8000-bd35-8d53b995c5a8

ChatGPT

A conversational AI system that listens, learns, and challenges

AA•10/4/24, 5:39 PM

it went a little sideways

lucasp•10/4/24, 8:04 PM

hello im getting this error, someone knows whats happend?

import { auth } from '@/auth'
import { getUser } from '@/client/services/user.service'
import Home from '@/components/pages/Home'
import { redirect } from 'next/navigation'

export const runtime = 'edge'

export default async function Main() {
  const session = await auth()

  const user = await getUser(session?.user?.localId ?? '')

  if (!user?.cargo_giver) redirect('/profile')

  return <Home />
}

import { auth } from '@/auth'
import { getUser } from '@/client/services/user.service'
import Home from '@/components/pages/Home'
import { redirect } from 'next/navigation'

export const runtime = 'edge'

export default async function Main() {
  const session = await auth()

  const user = await getUser(session?.user?.localId ?? '')

  if (!user?.cargo_giver) redirect('/profile')

  return <Home />
}

server side exception and when check logs

"logs": [
{
"message": [
"Error: The 'cache' field on 'RequestInitializerDict' is not implemented."
],

Spajk•10/4/24, 10:25 PM

This might be more of an nextjs issue then next-on-pages, but I randomly started getting this error:

import { auth } from '@/auth' import { getUser } from '@/client/services/user.service' import Home from '@/components/pages/Home' import { redirect } from 'next/navigation' export const runtime = 'edge' export default async function Main() { const session = await auth() const user = await getUser(session?.user?.localId ?? '') if (!user?.cargo_giver) redirect('/profile') return <Home /> }

it's the same

Similar Threads