Skip to content

Commit f582997

Browse files
committed
some typecheck fixes
1 parent 6217f14 commit f582997

File tree

2 files changed

+8
-3
lines changed

2 files changed

+8
-3
lines changed

rsconnect/api.py

+5-2
Original file line numberDiff line numberDiff line change
@@ -282,7 +282,7 @@ def fmt_payload(self) -> str:
282282
payload = urlencode(payload)
283283
return payload
284284

285-
def exchange_token(self) -> str | bytes:
285+
def exchange_token(self) -> str:
286286
try:
287287
server = HTTPServer(url=self.token_endpoint())
288288
payload = self.fmt_payload()
@@ -313,6 +313,9 @@ def exchange_token(self) -> str | bytes:
313313
if not response.response_body:
314314
raise RSConnectException("Token exchange returned empty response")
315315

316+
# Ensure we return a string
317+
if isinstance(response.response_body, bytes):
318+
return response.response_body.decode("utf-8")
316319
return response.response_body
317320

318321
except RSConnectException as e:
@@ -354,7 +357,7 @@ def __init__(self, server: PositConnectServer, cookies: Optional[CookieJar] = No
354357
if server.bootstrap_jwt:
355358
self.bootstrap_authorization(server.bootstrap_jwt)
356359

357-
if server.snowflake_connection_name:
360+
if server.snowflake_connection_name and isinstance(server, SPCSConnectServer):
358361
token = server.exchange_token()
359362
self.snowflake_authorization(token)
360363

rsconnect/main.py

+3-1
Original file line numberDiff line numberDiff line change
@@ -636,7 +636,9 @@ def list_servers(verbose: int):
636636
if server.get("ca_cert"):
637637
click.echo(" Client TLS certificate data provided")
638638
if server.get("snowflake_connection_name"):
639-
click.echo(' Snowflake Connection Name: "%s"' % server["snowflake_connection_name"])
639+
snowflake_connection_name = server.get("snowflake_connection_name")
640+
if snowflake_connection_name:
641+
click.echo(' Snowflake Connection Name: "%s"' % snowflake_connection_name)
640642
click.echo()
641643

642644

0 commit comments

Comments
 (0)