Skip to content

[EDPOPS-395] Remove API key option for login function #508

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

Merged
pfm-ramesh merged 1 commit into from
Jan 12, 2026
Merged

[EDPOPS-395] Remove API key option for login function #508

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
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
5 changes: 2 additions & 3 deletions recipes/sync_recipes.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,11 +8,10 @@
@click.group()
@click.option('--access-token', help='Manually provide a SolveBio Access Token')
@click.option('--api-host', help='Override the default SolveBio API host')
@click.option('--api-key', help='Manually provide a SolveBio API key')
@click.pass_context
def sync_recipes(ctx, api_key=None, access_token=None, api_host=None):
def sync_recipes(ctx, access_token=None, api_host=None):

sb.login(api_key=api_key, access_token=access_token, api_host=api_host,
sb.login(access_token=access_token, api_host=api_host,
version=__version__, name="SolveBio Recipes")
user = sb.User.retrieve()

Expand Down
6 changes: 0 additions & 6 deletions solvebio/__init__.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -168,7 +168,6 @@ def print_deprecation_notice():

def login(
api_host: str = None,
api_key: str = None,
access_token: str = None,
name: str = None,
version: str = None,
Expand All @@ -187,7 +186,6 @@ def login(
:param api_host: the QuartzBio EDP instance's URL to access.
:param access_token: your user's access token, which you can generate at the EDP website
(user menu > `Personal Access Tokens`)
:param api_key: Your API key. You can use this instead of providing an access token
:param name: name
:param version: version

Expand All @@ -197,7 +195,6 @@ def login(
import quartzbio
quartzbio.login(
api_host="https://solvebio.api.az.aws.quartz.bio",
api_key=YOUR_API_KEY
)
"""
token_type: Literal["Bearer", "Token"] = None
Expand All @@ -206,9 +203,6 @@ def login(
if access_token:
token_type = "Bearer"
token = access_token
elif api_key:
token_type = "Token"
token = api_key

if api_host or token or debug:
client.set_credentials(
Expand Down
2 changes: 0 additions & 2 deletions solvebio/cli/main.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -470,7 +470,6 @@ def __init__(self, *args, **kwargs):
help="Override the default SolveBio API host",
type=self.api_host_url,
)
self.add_argument("--api-key", help="Manually provide a SolveBio API key")
self.add_argument(
"--access-token", help="Manually provide a SolveBio OAuth2 access token"
)
Expand Down Expand Up @@ -534,7 +533,6 @@ def main():

solvebio.login(
api_host=args.api_host,
api_key=args.api_key,
access_token=args.access_token,
)

Expand Down
Loading