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.

Sign up for GitHub

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

GlobusApp login, logout, and login_required #1041

Merged
merged 6 commits into from
Sep 6, 2024
+199 −39
Merged

GlobusApp login, logout, and login_required #1041

Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
4cb523c
GlobusApp login, logout, and login_required
derek-globus Sep 4, 2024
ab7802d
Feedback: fixup mostly docstring wording
derek-globus Sep 4, 2024
6ddd3ba
Remove transfer-client references to app.run_login_flow()
derek-globus Sep 4, 2024
2c409e7
Feedback: list-form
derek-globus Sep 5, 2024
84844e8
Use public token_storage interface
derek-globus Sep 5, 2024
b4cfb33
Feedback: update docstrings
derek-globus Sep 6, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Loading
Loading status checks…
Feedback: list-form
@derek-globus
derek-globus committed Sep 5, 2024

Unverified

This commit is not signed, but one or more authors requires that any commit attributed to them is signed.
Learn about vigilant mode
commit 2c409e7ab983f880543a2a6ff36c239725b4d254
6 changes: 3 additions & 3 deletions src/globus_sdk/experimental/globus_app/app.py
View file
Original file line number Diff line number Diff line change
@@ -237,9 +237,9 @@ def login(
Log an auth entity into the app, if needed, storing the resulting tokens.

A login flow will be performed if:
* ``auth_params`` are provided
* ``force`` is set to True
* ``self.login_required()`` evaluates to True
* ``auth_params`` are provided,
* ``force`` is set to True, or
* ``self.login_required()`` evaluates to True.

:param auth_params: An optional set of authorization parameters to establish
requirements and controls for the login flow.