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

NOT_FOUND with spaces in path #944

Open
bogeychan opened this issue Dec 17, 2024 · 0 comments · May be fixed by #958
Open

NOT_FOUND with spaces in path #944

bogeychan opened this issue Dec 17, 2024 · 0 comments · May be fixed by #958
Labels
bug Something isn't working

Comments

@bogeychan
Copy link
Contributor

What version of Elysia is running?

1.1.26

What platform is your computer?

WSL Ubuntu

What steps can reproduce the bug?

// test.test.ts
import { test, expect } from "bun:test";
import { Elysia } from "elysia";

test("Spaces in path", async () => {
  const PATH = "/y a y";

  const app = new Elysia().get(PATH, "yay");

  const response = await app.handle(new Request(`http://localhost${PATH}`));

  expect(response.status).toBe(200);
});

What is the expected behavior?

test pass

What do you see instead?

test fails:

Expected: 200
Received: 404

Additional information

relates to elysiajs/elysia-static#35

Have you try removing the node_modules and bun.lockb and try again yet?

yes

@bogeychan bogeychan added the bug Something isn't working label Dec 17, 2024
@bogeychan bogeychan closed this as not planned Won't fix, can't repro, duplicate, stale Dec 18, 2024
@bogeychan bogeychan reopened this Dec 18, 2024
macabeus added a commit to macabeus/elysia that referenced this issue Dec 21, 2024
@macabeus macabeus linked a pull request Dec 21, 2024 that will close this issue
macabeus added a commit to macabeus/elysia that referenced this issue Dec 23, 2024
macabeus added a commit to macabeus/elysia that referenced this issue Dec 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant