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

send needs upgraded #8208

Closed
shatran opened this issue Aug 1, 2019 · 3 comments · Fixed by #8424
Closed

send needs upgraded #8208

shatran opened this issue Aug 1, 2019 · 3 comments · Fixed by #8424
Labels
good first issue Easy to fix issues, good for newcomers
Milestone

Comments

@shatran
Copy link

shatran commented Aug 1, 2019

Bug report

Describe the bug

When running ts-node -r tsconfig-paths/register src/server for a local development of a ts project, the following error occur:

Error: Attempt to change mapping for "wasm" extension from "application/wasm" to "application/wasm". Pass `force=true` to allow this, otherwise remove "wasm" from the list of extensions for "application/wasm".
    at Mime.define (.../node_modules/mime/Mime.js:54:15)
    at Object.<anonymous> (.../node_modules/next-server/dist/server/serve-static.js:8:21)

To Reproduce

Steps to reproduce the behavior, please provide code snippets or a repository:

  1. run ts-node with tsconfig-paths (to support tsconfig paths attributes, ts-node does not support it by default)

Expected behavior

Paths support to a ts project should be allowed. I would expect to be able to run ts-node together with tsconfig-paths

System information

  • OS: macOS
  • Version of Next.js: 9.0.3-canary.1

Additional context

Iv'e raised a PR to solve this issue: #8209

@Timer
Copy link
Member

Timer commented Aug 11, 2019

Hey! This is likely due to some sort of double-evaluation, however, we cannot know that for sure because a reproducible demo was not provided.

Regarding this specific error, it sounds like upgrading send to its latest version is the best approach, then removing our line that defines a wasm type.

Can you please send a PR doing this?

@Timer Timer added good first issue Easy to fix issues, good for newcomers help wanted and removed Type: Needs Investigation labels Aug 11, 2019
@Timer Timer changed the title next-server fails to run with ts-node using tsconfig-paths send needs upgraded Aug 14, 2019
Timer added a commit to Timer/next.js that referenced this issue Aug 19, 2019
This upgrade adds native support for wasm.

---

Closes vercel#8208
timneutkens pushed a commit that referenced this issue Aug 19, 2019
This upgrade adds native support for wasm.

---

Closes #8208
@Timer
Copy link
Member

Timer commented Aug 19, 2019

Fixed in 9.0.4.

@balazsorban44
Copy link
Member

This issue has been automatically locked due to no recent activity. If you are running into a similar issue, please create a new issue with the steps to reproduce. Thank you.

@vercel vercel locked as resolved and limited conversation to collaborators Jan 31, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
good first issue Easy to fix issues, good for newcomers
Projects
None yet
3 participants