Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WIP: Support server-side debugging when running local in docker #450

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

SatsAllDay
Copy link
Contributor

Update ports exposed, and run the next server with inspect flag

Currently does not work due to bug in next.js
See related PR:
vercel/next.js#53683

Leaving this in draft mode until next.js fixes their issue.

Update ports exposed, and run the next server with inspect flag

Currently does not work due to bug in next.js
See related PR:
vercel/next.js#53683
@ekzyis ekzyis added the sndev label Sep 7, 2023
@SatsAllDay
Copy link
Contributor Author

Finally some activity on the aforementioned nextjs PR. Maybe this will be doable soon :)

@huumn
Copy link
Member

huumn commented Oct 23, 2023

ek and I were talking about it this morning! 🙌

@SatsAllDay
Copy link
Contributor Author

Woo hoo! Upstream support has merged vercel/next.js#65006

Now we wait for it to be GA'd and then we can bump nextjs dependency versions and then we can debug!

@huumn huumn marked this pull request as ready for review October 15, 2024 23:39
@huumn
Copy link
Member

huumn commented Oct 16, 2024

Looks like this still hasn't been cherry picked into v14.2. Well dang.

@SatsAllDay
Copy link
Contributor Author

Big sad

@huumn
Copy link
Member

huumn commented Nov 8, 2024

On v14.2.17, it's still not picked in.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants