Skip to content
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.

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

add rn entry for refreshing access token after snowflake sso expiration #6790

Merged
merged 3 commits into from
Jan 17, 2025

Conversation

mirnawong1
Copy link
Contributor

@mirnawong1 mirnawong1 commented Jan 17, 2025

this pr adds a rn entry that adds an enhancment for the SL:

added support for refreshing access token for Snowflake SSO connections when it expires

Linear: https://linear.app/dbt-labs/issue/SL-2264/support-refreshing-access-token-for-snowflake-sso-connections-when-it


🚀 Deployment available! Here are the direct links to the updated files:

@mirnawong1 mirnawong1 requested a review from a team as a code owner January 17, 2025 09:41
Copy link

vercel bot commented Jan 17, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
docs-getdbt-com ✅ Ready (Inspect) Visit Preview Jan 17, 2025 5:14pm

@github-actions github-actions bot added content Improvements or additions to content Docs team Authored by the Docs team @dbt Labs size: x-small This change will take under 3 hours to fix. and removed Docs team Authored by the Docs team @dbt Labs labels Jan 17, 2025
@@ -19,6 +19,8 @@ Release notes are grouped by month for both multi-tenant and virtual private clo
\* The official release date for this new format of release notes is May 15th, 2024. Historical release notes for prior dates may not reflect all available features released earlier this year or their tenancy availability.

## January 2025

- **Enhancement**: Added support to automatically refresh access tokens when Snowflake's SSO connection expires. Previously, you had to manually refresh the token when the connection expired, resulting in downtime for your dbt Cloud Semantic Layer connection.
Copy link
Contributor

Choose a reason for hiding this comment

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

Hiya @mirnawong1

This is a non-blocker suggestion.
Would it be a good idea to add a sentence saying something like the enhancement improves / enables better performance when using the SL?

Kind Regards
Natalie

Choose a reason for hiding this comment

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

I would replace the "previously" part with something like:

Previously, users would get the following error: Connection is not available, request timed out after 30000ms and would have to wait 10 minutes to try again.

Copy link
Contributor

@nataliefiann nataliefiann left a comment

Choose a reason for hiding this comment

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

Hiya @mirnawong1

Thanks for creating this PR. I've approved this for you and left a question / non blocking suggestion.

Kind Regards
Natalie

@github-actions github-actions bot added the Docs team Authored by the Docs team @dbt Labs label Jan 17, 2025
@mirnawong1 mirnawong1 enabled auto-merge January 17, 2025 17:06
@mirnawong1 mirnawong1 merged commit 03385ab into current Jan 17, 2025
8 checks passed
@mirnawong1 mirnawong1 deleted the snowflake-sso-refresh-rn branch January 17, 2025 17:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content Improvements or additions to content Docs team Authored by the Docs team @dbt Labs size: x-small This change will take under 3 hours to fix.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants