Skip to content

Conversation

@cemalkilic
Copy link
Contributor

Summary

This PR adds support for configurable OAuth authorization TTL (Time To Live) using theGOTRUE_OAUTH_SERVER_AUTHORIZATION_TTL environment variable, replacing the previous hardcoded 10-minute expiration. This is safe & requires no migration as it's not rolled out yet!

Also introduces params struct to be used with NewOAuthServerAuthorization.

@coveralls
Copy link

coveralls commented Nov 12, 2025

Pull Request Test Coverage Report for Build 19453828995

Details

  • 15 of 26 (57.69%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.004%) to 68.354%

Changes Missing Coverage Covered Lines Changed/Added Lines %
internal/api/oauthserver/authorize.go 0 11 0.0%
Totals Coverage Status
Change from base Build 19453720722: -0.004%
Covered Lines: 14351
Relevant Lines: 20995

💛 - Coveralls

Copy link
Contributor

@issuedat issuedat left a comment

Choose a reason for hiding this comment

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

🚀

@cemalkilic cemalkilic force-pushed the cemal/feat-authorization-ttl-envvar branch from ca62e42 to 4559346 Compare November 18, 2025 04:11
@cemalkilic cemalkilic merged commit 61632f8 into master Nov 18, 2025
5 checks passed
@cemalkilic cemalkilic deleted the cemal/feat-authorization-ttl-envvar branch November 18, 2025 04:27
cemalkilic pushed a commit that referenced this pull request Nov 24, 2025
🤖 I have created a release *beep* *boop*
---


##
[2.183.0](v2.182.1...v2.183.0)
(2025-11-20)


### Features

* async, concurrent index creation for users table
([#2239](#2239))
([a1146bf](a1146bf))
* **indexworker:** use `auth_trgm` extension if available
([#2263](#2263))
([05daa43](05daa43))
* **oauthserver:** add OpenID Connect support
([#2250](#2250))
([162788f](162788f))
* **oauthserver:** update oauth grant list & authorization details
response structure
([#2247](#2247))
([137ea92](137ea92))
* **oauthserver:** use `NewOAuthServerAuthorizationParams` &
configurable ttl for authorization
([#2254](#2254))
([61632f8](61632f8))


### Bug Fixes

* **indexworker:** detect which schema `pg_trgm` exists in
([#2260](#2260))
([4be12b3](4be12b3))
* look for refresh token on mfa verification only in v1
([#2249](#2249))
([2906b24](2906b24))
* mfa verify now works with refresh token algorithm v2
([#2246](#2246))
([4e8275f](4e8275f))
* **social-auth:** default to current_user:read for Figma provider
([#2195](#2195))
([f409d11](f409d11))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

4 participants