Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Deprecate LoginManager and AuthorizerLoginManager #1791

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

rjmello
Copy link
Member

@rjmello rjmello commented Feb 5, 2025

Description

  • Deprecated the LoginManager and AuthorizerLoginManager classes.
  • Deprecated the Client.login_manager attribute.
  • Added an authorizer parameter to the Client initializer to enable using a GlobusAuthorizer for authentication.
  • The SDK docs on using an existing access token now instruct users to pass in a globus_sdk.AccessTokenAuthorizer to the Client initializer instead of using the deprecated AuthorizerLoginManager.

[sc-38908]

Type of change

  • New feature (non-breaking change that adds functionality)
  • Code maintenance/cleanup

`GlobusApp` objects should be used instead of `LoginManager` objects.
The `authorizer` parameter enables using a ``GlobusAuthorizer`` for
authentication and is mutually exclusive with `app`. This also enables
users to replace the deprecated `AuthorizerLoginManager` functionality
by directly passing in an `AccessTokenAuthorizer` from the Globus SDK.
The SDK docs on using an existing access token now instruct users to
pass in a `globus_sdk.AccessTokenAuthorizer` to the `Client` initializer
instead of using the deprecated `AuthorizerLoginManager`.
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.

1 participant