Skip to content

Render 429 error page for Turbo frame requests#8463

Merged
iHiD merged 1 commit into
mainfrom
fix/8450
Feb 9, 2026
Merged

Render 429 error page for Turbo frame requests#8463
iHiD merged 1 commit into
mainfrom
fix/8450

Conversation

@iHiD

@iHiD iHiD commented Feb 9, 2026

Copy link
Copy Markdown
Member

Closes #8450

Summary

  • Turbo frame requests are XHR, so the rate limit concern was returning head :too_many_requests (empty body) for them
  • Turbo then couldn't find <turbo-frame id="tf-main"> in the empty response and threw an error (the Sentry issue)
  • More importantly, the user saw nothing — a silent failure with no explanation
  • Now Turbo frame requests fall through to render the styled 429 error page inside the turbo_frame layout, giving users a proper "too many requests" message
  • Also adds the missing status: :too_many_requests to the render call (was defaulting to 200)

Test plan

  • Controller tests pass (0 failures)
  • Pre-commit hooks pass

🤖 Generated with Claude Code

When rate-limited users navigate via Turbo frames, the concern returned
`head :too_many_requests` (empty body) because Turbo frame requests are
XHR. Turbo then failed to find `<turbo-frame id="tf-main">` and threw
an error. Now Turbo frame requests render the styled 429 error page
inside the turbo_frame layout, giving users a proper explanation.

Also adds the missing `status: :too_many_requests` to the render call.

Closes #8450

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@iHiD iHiD merged commit fe08a56 into main Feb 9, 2026
41 checks passed
@iHiD iHiD deleted the fix/8450 branch February 9, 2026 10:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

1 participant