Skip to content

feat(auth): add OAuth server configuration support - #4207

Merged
cemalkilic merged 6 commits into
developfrom
cemal/feat-support-oauth-server
Sep 25, 2025
Merged

feat(auth): add OAuth server configuration support#4207
cemalkilic merged 6 commits into
developfrom
cemal/feat-support-oauth-server

Conversation

@cemalkilic

@cemalkiliccemalkilic commented Sep 25, 2025

Copy link
Copy Markdown
Contributor

Summary

Add support for new OAuth server environment variables in local development:

  • GOTRUE_OAUTH_SERVER_ENABLED
  • GOTRUE_OAUTH_SERVER_ALLOW_DYNAMIC_REGISTRATION
  • GOTRUE_OAUTH_SERVER_AUTHORIZATION_PATH

The feature isn't in GA yet in the platform, so skipping updating the remote config for now.

@cemalkilic
cemalkilic requested a review from a team as a code ownerSeptember 25, 2025 09:48
@coveralls

coveralls commented Sep 25, 2025•
edited
Loading

Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 18006692323

Details

  • 2 of 8(25.0%) changed or added relevant lines in 1 file are covered.
  • 5 unchanged lines in 1 file lost coverage.
  • Overall coverage decreased (-0.05%) to 54.888%
Changes Missing CoverageCovered LinesChanged/Added Lines%
internal/start/start.go2825.0%
Files with Coverage ReductionNew Missed Lines%
internal/gen/keys/keys.go512.9%
TotalsCoverage Status
Change from base Build 18004365792:-0.05%
Covered Lines:6367
Relevant Lines:11600
💛 - Coveralls

sweatybridge
sweatybridge reviewed Sep 25, 2025
View reviewed changes

@sweatybridgesweatybridge left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

just some nitpick

Comment threadpkg/config/templates/config.toml Outdated
Comment threadpkg/config/templates/config.toml
Comment threadinternal/start/start.go Outdated
@cemalkilic
cemalkilic merged commit 8d577b7 into developSep 25, 2025
15 checks passed
@cemalkilic
cemalkilic deleted the cemal/feat-support-oauth-server branch September 25, 2025 13:51
@github-actionsgithub-actionsBot mentioned this pull request Sep 29, 2025
Merged
@neuromusicneuromusic mentioned this pull request Aug 27, 2026
Open
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sweatybridgesweatybridgesweatybridge approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@cemalkilic@coveralls@sweatybridge