Skip to content

Catch-all Route Doesn't Support Special Characters #2012

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

Closed
knpwrs opened this issue Feb 17, 2022 · 4 comments
Closed

Catch-all Route Doesn't Support Special Characters #2012

knpwrs opened this issue Feb 17, 2022 · 4 comments
Labels
bug Something isn't working

Comments

@knpwrs
Copy link

knpwrs commented Feb 17, 2022

What version of Remix are you using?

1.1.3

Steps to Reproduce

Create a catch-all route handler at app/routes/$.ts.

Expected Behavior

All routes should be handled by app/routes/$.ts.

Actual Behavior

Routes like <host>/foo and <host>/foo/bar work, but routes like <host>/~foo and <host>/@foo/bar return 404.

@knpwrs knpwrs added the bug Something isn't working label Feb 17, 2022
@knpwrs
Copy link
Author

knpwrs commented Feb 17, 2022

Note that encoding @ as %40 leads to the same result.

@knpwrs
Copy link
Author

knpwrs commented Feb 18, 2022

I attempted to create a file called @$.ts as a workaround, but it did not work:

export { loader } from './$';

@knpwrs
Copy link
Author

knpwrs commented Feb 20, 2022

This appears to be a bug with React Router: remix-run/react-router#8525

@chaance
Copy link
Collaborator

chaance commented Mar 16, 2022

This is currently as designed. Note remix-run/react-router#8525 (comment), but we may yet be convinced change this behavior. In any event, this is related to React Router so I'm going to close this and refer folks to comment in the linked issue there instead.

@chaance chaance closed this as completed Mar 16, 2022
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

No branches or pull requests

2 participants