Skip to content

Unify login return redirects - #2059

Merged
yusuftomilola merged 9 commits into
DistinctCodes:mainfrom
ObedChibunna:main
Oct 20, 2026
Merged

yusuftomilola merged 9 commits into
DistinctCodes:mainfrom
ObedChibunna:main

Conversation

@ObedChibunna

@ObedChibunna ObedChibunna commented Oct 18, 2026 •

Copy link
Copy Markdown
Contributor

Use only the validated returnTo parameter across middleware, admin guards, and the auth form. Preserve the full path and query string when redirecting after login, and remove all production of next.

CLOSES #1993

Summary

Use only the validated returnTo parameter across middleware, admin guards, and the auth form. Preserve the full path and query string when redirecting after login, and remove all production of next.

Contracts checklist

If this PR touches anything under contracts/, confirm it against the
pre-deployment security checklist before
requesting review:

  • make audit (CT-72 require_auth() coverage) passes.
  • Every amount-handling path uses checked arithmetic (CT-73) — no raw
    +/- on amounts.
  • Not applicable — this PR does not touch contracts/.

Test plan

Frontend CI checks passed.
Backend build, lint, dependency checks, critical audit, and unit tests passed.
Database-backed checks and Rust build checks remain unverified locally: PostgreSQL was unavailable, and the pinned Rust installation ran out of disk space.

Verified frontend lint, unit tests, typecheck, build, and E2E. Backend build, unit tests (714 passed), lint, dependency checks, and critical audit passed. Database checks and Rust builds remain unverified: PostgreSQL was unavailable, and installing the pinned Rust toolchain ran out of disk space.

CLOSES #1993

Use only the validated returnTo parameter across middleware, admin guards, and the auth form. Preserve the full path and query string when redirecting after login, and remove all production of next.

CLOSES DistinctCodes#1993
@vercel

vercel Bot commented Oct 18, 2026

Copy link
Copy Markdown

@ObedChibunna is attempting to deploy a commit to the naijabuz's projects Team on Vercel.

A member of the Team first needs to authorize it.

@drips-wave

drips-wave Bot commented Oct 18, 2026

Copy link
Copy Markdown

@ObedChibunna Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@yusuftomilola
yusuftomilola merged commit b096a0c into DistinctCodes:main Oct 20, 2026
7 of 8 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.

The login next and returnTo parameters are both set and both read

2 participants