-
Notifications
You must be signed in to change notification settings - Fork 86
basePath
option is not working
#131
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
Comments
hey, @vitorbal! do you have a reproduction available for us to review? and/or the build logs for the failing deploy? that helps us track things down much more quickly thank you! |
@vitorbal does this forum post help? |
@jlengstorf thanks!
as you can see |
@jdeal thanks for sharing that forum post! It sort of helps, but in their case, they are deploying an app via |
hey, @vitorbal — I think I know the issue here. with
I set up an example of this:
source code:
let me know if this makes sense. thanks! |
hey @vitorbal! did jason's comment solve your issue? let us know so we can close the issue or continue to try to help! :D |
Howdy folks! Apologies for the lul. @jlengstorf I checked out your app and even though it solves the problem, I would expect a standalone next.js app to work without those redirects. If you run your next app locally, the If it helps, I deployed the reproduction case to Vercel and it also works as expected there: https://next-basepath.vercel.app/foo |
@vitorbal @jlengstorf I've just run in to this too. I don't think redirecting from root to a subpath is the right solution. When I run |
@lindsaylevine @jlengstorf sounds good, thank you both for being so responsive! |
* ci: run tests on canary and skip none * DIsable one so script works * Use manifest v2 * Remove old filter step * Fix versions * Exclude invalid tests * Exclude more * Use junit * Set cwd for test report * Try massive concurrency * Fix concurrency * Use different test reporter * Adjust condiitons * Typo * Fix junit path * Disable more * fix: shim process in edge runtime * Switch reporter action * ci: fix artifact path * Disable very slow tests * Add test runner * Use md formatter * Add deno install step to task * Fix tool path * Disable test * Skip bad tests * Omit non-deploy test * Skip tests that are failing because of whitespace in headers * Continute on error in local tests * Skip a non-deploy test * Skip xff test * Skip tests that try to patch files * Checkout runtime in report step * DIsable more flaky tests * Fix script name * Log cast totals * Format better
Describe the bug
If you set up a very simple nextjs app with the
basePath
option configured, the deployed website will 404 a bunch of static asset paths. Here's an example for abasePath
of/cats-of-zapier
:To Reproduce
Steps to reproduce the behavior:
basePath: '/foo'
tonext.config.js
Expected behavior
For the app to be served correctly, with no 404s.
Versions
The text was updated successfully, but these errors were encountered: