Skip to content

Conversation

@edavidaja
Copy link
Collaborator

initial support for interacting with Connect in SPCS

Intent

Type of Change

  • Bug Fix
  • New Feature
  • Breaking Change

Approach

Take an optional dependency on the snow CLI
commands requiring interaction with remote servers gain an optional --snowflake-connection-name argument
The ingress URL, account name, and keypair configured out of band are sufficient information
to obtain the OAuth token required to interact with Connect in SPCS.

supports rsconnect add, details, and deploy_html

Automated Tests

Directions for Reviewers

Checklist

  • I have updated CHANGELOG.md to cover notable changes.
  • I have updated all related GitHub issues to reflect their current state.

# This is specifically not None.
self.cookie_jar = CookieJar()
# 🤡
self.snowflake_connection_name = None
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Curious as to why this is needed here but not in the other non SPCS targetable servers.

pass a --snowflake-connection-name argument to the CLI, which uses
the snow cli to obtain connection details and perform token exchange
on each call to connect

supports rsconnect add, details, and deploy_html
@edavidaja edavidaja closed this Apr 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants