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

Does not work with trailingSlash: true #9

Open
matusfaro opened this issue Dec 19, 2023 · 1 comment
Open

Does not work with trailingSlash: true #9

matusfaro opened this issue Dec 19, 2023 · 1 comment

Comments

@matusfaro
Copy link

This doesn't necessarily need a fix, but I stumbled upon this in my deployment: you can't use trailingSlash: true in your nextConfig as it's incompatible with the url rewriter CloudFront function that relies on static pages being as a/b.html rather than a/b/index.html.

@dkershner6
Copy link
Owner

This is true, I specifically avoided supporting this as building because it would actually use a significant amount more logic in the loop, causing possible cloudfront function timeouts (which was a major problem during beta versions) and shrink the number of pages allowed (due to code size).

Happy to be proven wrong here, if someone wants to give it a shot, but I am wary of success.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants