Skip to content

Conversation

@pan93412
Copy link
Member

No description provided.

@pan93412 pan93412 self-assigned this Aug 24, 2025
@linear
Copy link

linear bot commented Aug 24, 2025

@pan93412 pan93412 requested a review from Copilot August 24, 2025 17:06

This comment was marked as outdated.

@pan93412 pan93412 requested a review from Copilot August 24, 2025 17:14
Copy link
Contributor

Copilot AI left a comment

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 enhances the Google OAuth authentication system to support multiple redirect URIs instead of a single redirect URL, improving flexibility for multi-domain applications.

  • Changed configuration from single RedirectURL to multiple RedirectURIs
  • Added redirect URI validation logic in the callback handler
  • Implemented cookie-based redirect URI storage and retrieval

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
internal/config/models.go Updates GAuthConfig to use slice of redirect URIs instead of single URL
httpapi/auth/root.go Updates handler initialization to pass redirect URIs array
httpapi/auth/gauth.go Implements redirect URI validation and cookie-based redirect handling
httpapi/auth/README.md Updates documentation to mention redirect_uri parameter
docs/config.md Updates configuration documentation for multiple redirect URIs

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@pan93412 pan93412 merged commit 83bd337 into main Aug 24, 2025
1 check passed
@pan93412 pan93412 deleted the pan93412/dbp-10-allow-multiple-redirect-uris branch September 14, 2025 11:00
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