Fix OAuth redirectURL by darioalessandro · Pull Request #485 · security-union/videocall-rs · GitHub
Skip to content

Fix OAuth redirectURL - #485

Merged
darioalessandro merged 1 commit into
mainfrom
fix-oauth-links
Nov 13, 2025
Merged

darioalessandro merged 1 commit into
mainfrom
fix-oauth-links

Conversation

@darioalessandro

@darioalessandro darioalessandro commented Nov 13, 2025 •

Copy link
Copy Markdown
Member

Fixes #481


Note

Adds end-to-end returnTo support across OAuth (store in DB, use on callback) and UI (pass/encode current URL), plus enables WebTransport by default.

  • Auth/Backend:
    • Accepts returnTo in /login (LoginQuery) and persists it with OAuth request via generate_and_store_oauth_request.
    • DB migration: add oauth_requests.return_to column.
    • Includes return_to in OAuthRequest and reads it in fetch_oauth_request.
    • After callback, redirects to stored return_to or falls back to after_login_url (default /).
  • UI:
    • components/login.rs: appends current URL query to backend login URL.
    • pages/meeting.rs: on no session, redirects to /login?returnTo=<encoded current URL>; adds urlencoding dependency.
    • scripts/config.js: sets webTransportEnabled to true.
  • Diagnostics:
    • Minor wasm receiver loop condition cleanup to keep broadcast channel alive.

Written by Cursor Bugbot for commit e50fbbc. This will update automatically on new commits. Configure here.

@cursor cursor Bot 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.

This is the final PR Bugbot will review for you during this billing cycle

Your free Bugbot reviews will reset on December 31

Details

You are on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

Comment thread yew-ui/scripts/config.js

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Bug: Accidental Dev Config Setting Commit

The webTransportEnabled setting was changed from "false" to "true" in the development config file. This appears unrelated to the OAuth redirect URL fix described in the PR title and may be an accidentally committed local development configuration change.

Fix in Cursor Fix in Web

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@darioalessandro
darioalessandro merged commit e07d5ce into main Nov 13, 2025
2 checks passed
@darioalessandro
darioalessandro deleted the fix-oauth-links branch November 13, 2025 02:35
This was referenced Nov 13, 2025
@testradav

Copy link
Copy Markdown
Collaborator

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.

We loose the meeting link with oauth

2 participants