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

Too Many Redirects error with ISR and optional route params #10155

Closed
adrienpicard opened this issue Jun 14, 2023 · 2 comments · Fixed by #11928
Closed

Too Many Redirects error with ISR and optional route params #10155

adrienpicard opened this issue Jun 14, 2023 · 2 comments · Fixed by #11928
Labels
pkg:adapter-vercel Pertaining to the Vercel adapter

Comments

@adrienpicard
Copy link

adrienpicard commented Jun 14, 2023

Describe the bug

Possibly related to #9459 but I'm receiving a "Too Many Redirects" error when using ISR with the Vercel adapter.

The route structure is /[first]/[[second]] and I'm getting the error when directly visiting URLs with only the first parameter.

Reproduction

https://kit-isr-demo.vercel.app
https://github.com/adrienpicard/kit-isr-demo

Navigating to /lorem or /ipsum from the homepage works but not if you access the page directly or hard refresh.

Logs

---

System Info

System:
    OS: macOS 12.6
    CPU: (10) arm64 Apple M1 Pro
    Memory: 97.77 MB / 16.00 GB
    Shell: 5.8.1 - /bin/zsh
  Binaries:
    Node: 16.18.1 - ~/.nvm/versions/node/v16.18.1/bin/node
    Yarn: 1.22.19 - /opt/homebrew/bin/yarn
    npm: 8.19.2 - ~/.nvm/versions/node/v16.18.1/bin/npm
  Browsers:
    Chrome: 114.0.5735.106
    Firefox: 114.0.1
    Safari: 16.4
  npmPackages:
    @sveltejs/adapter-auto: ^2.0.0 => 2.1.0 
    @sveltejs/adapter-vercel: ^3.0.1 => 3.0.1 
    @sveltejs/kit: ^1.5.0 => 1.20.2 
    svelte: ^3.54.0 => 3.59.1 
    vite: ^4.3.0 => 4.3.9

Severity

blocking all usage of SvelteKit

Additional Information

No response

@pdrbrnd
Copy link

pdrbrnd commented Oct 4, 2023

@adrienpicard did you manage to get it working?

I know this is a relatively old issue but I have just came across the exact same problem.
Usually the team is quite responsive so I find it strange that there is no activity here.

@adrienpicard
Copy link
Author

adrienpicard commented Oct 5, 2023

@pdrbrnd sadly not, I went with another directory/page structure until this is issue is resolved.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pkg:adapter-vercel Pertaining to the Vercel adapter
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants