Skip to content

Commit 834d1b6

Browse files
committed
fix(idp, BrowserAzureCredentialsProvider): remove client_secret parameter
1 parent 86c6cc4 commit 834d1b6

File tree

2 files changed

+85
-90
lines changed

2 files changed

+85
-90
lines changed

redshift_connector/plugin/browser_azure_credentials_provider.py

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,6 @@ class BrowserAzureCredentialsProvider(SamlCredentialsProvider):
2424
def __init__(self: "BrowserAzureCredentialsProvider") -> None:
2525
super().__init__()
2626
self.idp_tenant: typing.Optional[str] = None
27-
self.client_secret: typing.Optional[str] = None
2827
self.client_id: typing.Optional[str] = None
2928

3029
self.idp_response_timeout: int = 120
@@ -47,7 +46,6 @@ def add_parameter(self: "BrowserAzureCredentialsProvider", info: RedshiftPropert
4746
# The value of parameter idp_tenant.
4847
self.idp_tenant = info.idp_tenant
4948
# The value of parameter client_id.
50-
self.client_secret = info.client_secret
5149
self.client_id = info.client_id
5250

5351
self.idp_response_timeout = info.idp_response_timeout
@@ -122,7 +120,6 @@ def fetch_saml_response(self: "BrowserAzureCredentialsProvider", token):
122120
"scope": "openid",
123121
"resource": self.client_id,
124122
"client_id": self.client_id,
125-
"client_secret": self.client_secret,
126123
"redirect_uri": self.redirectUri,
127124
}
128125

0 commit comments

Comments
 (0)