Skip to content

Prevent Render warmup rate-limit failures during Xaman login - #11

Merged
xrpbanks merged 2 commits into
mainfrom
fix-xaman-cross-browser-return
Aug 29, 2026
Merged

Prevent Render warmup rate-limit failures during Xaman login#11
xrpbanks merged 2 commits into
mainfrom
fix-xaman-cross-browser-return

Conversation

@xrpbanks

Copy link
Copy Markdown
Contributor

Summary

  • replace the aggressive 2-second health polling loop with exponential backoff
  • respect Render Retry-After guidance and add a conservative fallback for HTTP 429 responses
  • extend the free-demo warmup window to three minutes and tell users not to refresh repeatedly
  • clarify that mobile Xaman may return in the phone's default browser
  • add a regression test proving a login started in one browser can finish securely in another

Verified locally

  • 201 backend tests passed
  • frontend lint passed
  • frontend production build and TypeScript validation passed
  • legal/licensing, whitespace, and tracked-artifact release checks passed

This PR does not change WordPress content or deploy anything by itself.

Copilot AI lite review requested due to automatic review settings August 29, 2026 00:33

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR improves resilience of the Xaman login flow when the backend is warming up on Render, reducing the likelihood of rate-limit failures and making the mobile return-to-browser behavior clearer.

Changes:

  • Replaces the fixed 2s backend warmup polling loop with exponential backoff and explicit HTTP 429 (Retry-After) handling.
  • Extends the warmup window to 3 minutes and updates user-facing messaging to discourage repeated refreshes.
  • Adds a regression test ensuring the identity callback can complete in a different browser session than the one that initiated login.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
frontend/lib/backendRequest.ts Implements exponential backoff warmup probing and respects rate-limit retry guidance; updates timeout and error copy.
frontend/components/XamanLoginPanel.tsx Updates UI messaging for longer warmup and cross-browser/mobile return behavior.
backend/tests/test_identity_endpoints.py Adds a regression test verifying callback completion in a different “browser” (separate client cookies).

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread frontend/lib/backendRequest.ts
@xrpbanks
xrpbanks merged commit d91fad3 into main Aug 29, 2026
4 checks passed
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

Successfully merging this pull request may close these issues.

2 participants