Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@stacurry
Copy link
Contributor

Description

  • adds support for the oauth_tokens object in the authenticate API response
  • adds support for the new provider_scopes query parameter

Documentation

Does this require changes to the WorkOS Docs? E.g. the API Reference or code snippets need updates.

[X] Yes

If yes, link a related docs PR and add a docs maintainer as a reviewer. Their approval is required.

@stacurry stacurry requested a review from a team as a code owner June 19, 2025 02:11
@stacurry stacurry requested a review from jonatascastro12 June 19, 2025 02:11
@linear
Copy link

linear bot commented Jun 19, 2025

@stacurry stacurry changed the title Auth 4667 custom scopes add support for custom scopes Jun 19, 2025
Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

PR Summary

Introduces OAuth token support and provider-specific scopes to the WorkOS PHP SDK, enhancing authentication flexibility and control.

  • Added new lib/Resource/OAuthTokens.php class to handle OAuth token responses including access tokens, refresh tokens, and expiration times
  • Added providerScopes parameter to getAuthorizationUrl in lib/UserManagement.php for fine-grained OAuth permission control
  • Updated lib/Resource/AuthenticationResponse.php to include OAuth tokens in authentication responses
  • Added comprehensive test coverage in tests/WorkOS/UserManagementTest.php for new OAuth features

4 files reviewed, 3 comments
Edit PR Review Bot Settings | Greptile

@stacurry stacurry force-pushed the auth-4667-custom-scopes branch from 77d3a21 to b9d5999 Compare June 19, 2025 02:30
@stacurry stacurry merged commit 2fa86c3 into main Jun 23, 2025
5 checks passed
@stacurry stacurry deleted the auth-4667-custom-scopes branch June 23, 2025 19:46
@stacurry stacurry mentioned this pull request Jun 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants