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

[Turbopack] load wrong assetPrefix (it uses http://localhost:3000/http://cdn.com) #60891

Closed
1 task done
thasuohm opened this issue Jan 19, 2024 · 1 comment · Fixed by #61090
Closed
1 task done

[Turbopack] load wrong assetPrefix (it uses http://localhost:3000/http://cdn.com) #60891

thasuohm opened this issue Jan 19, 2024 · 1 comment · Fixed by #61090
Labels
bug Issue was opened via the bug report template. linear: turbopack Confirmed issue that is tracked by the Turbopack team. locked Turbopack Related to Turbopack with Next.js.

Comments

@thasuohm
Copy link

thasuohm commented Jan 19, 2024

Link to the code that reproduces this issue

https://github.com/thasuohm/nextjs-reproduce/tree/turbopack/wrong-assetprefix-load

To Reproduce

  1. Create a simple next js app
  2. add assetPrefix in next.config.js
  3. run in dev mode with --turbo

Current vs. Expected behavior

AS: It loads assetPrefix with localhost as a prefix
image

EXPECTED: it loads correctly assetPrefix when run with --turbo

Verify canary release

  • I verified that the issue exists in the latest Next.js canary release

Provide environment information

Operating System:
  Platform: win32
  Arch: x64
  Version: Windows 10 Pro
Binaries:
  Node: 20.10.0
  npm: N/A
  Yarn: N/A
  pnpm: 7.25.0
Relevant Packages:
  next: 14.1.1-canary.1
  eslint-config-next: N/A
  react: 18.2.0
  react-dom: 18.2.0
  typescript: 5.1.3
Next.js Config:
  output: N/A

Which area(s) are affected? (Select all that apply)

App Router, Turbopack (--turbo)

Which stage(s) are affected? (Select all that apply)

next dev (local)

Additional context

This has happened Since version 14.0.3

PACK-2282

@thasuohm thasuohm added the bug Issue was opened via the bug report template. label Jan 19, 2024
@github-actions github-actions bot added the Turbopack Related to Turbopack with Next.js. label Jan 19, 2024
@ForsakenHarmony ForsakenHarmony added the linear: turbopack Confirmed issue that is tracked by the Turbopack team. label Jan 23, 2024
ForsakenHarmony added a commit that referenced this issue Jan 24, 2024
### Turbopack Updates

* vercel/turborepo#7091 <!-- Tobias Koppers - lock
reqwest to 0.11.17 due to build issues -->
* vercel/turborepo#7080 <!-- Tobias Koppers - reduce
work and tasks during module analysis -->
* vercel/turborepo#7093 <!-- Leah - fix: don't
escape base path in ecmascript runtime to allow for URLs -->

Closes PACK-2282
Fixes #60891 via vercel/turborepo#7093
Copy link
Contributor

github-actions bot commented Feb 8, 2024

This closed issue has been automatically locked because it had no new activity for 2 weeks. If you are running into a similar issue, please create a new issue with the steps to reproduce. Thank you.

@github-actions github-actions bot added the locked label Feb 8, 2024
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 8, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue was opened via the bug report template. linear: turbopack Confirmed issue that is tracked by the Turbopack team. locked Turbopack Related to Turbopack with Next.js.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants