Skip to content

Commit

Permalink
Improve comment.
Browse files Browse the repository at this point in the history
  • Loading branch information
VersusFacit committed Jan 15, 2025
1 parent dba6a01 commit 55f555a
Showing 1 changed file with 3 additions and 4 deletions.
7 changes: 3 additions & 4 deletions dbt/adapters/redshift/connections.py
Original file line number Diff line number Diff line change
Expand Up @@ -329,12 +329,11 @@ def __iam_idc_browser_kwargs(credentials) -> Dict[str, Any]:

def __iam_idc_token_kwargs(credentials) -> Dict[str, Any]:
"""
Expand the logic and flow control here to account for other Identity providers
with their specific request configurations. May require new parameters for
token_endpoint extraction to distinguish idp's if the url is insufficient.
Accepts a `credentials` object with a `token_endpoint` field that corresponds to
either Okta or Entra authentication services.
We only support token_type=EXT_JWT tokens. token_type=ACCESS_TOKEN has not been
tested or is supported but can be added with a presenting use-case.
tested. It can be added with a presenting use-case.
"""

logger.debug("Connecting to Redshift with '{credentials.method}' credentials method")
Expand Down

0 comments on commit 55f555a

Please sign in to comment.