Skip to content

Add Snowflake per-user OAuth docs - #2463

Merged
mustafa3rsan merged 4 commits into
mainfrom
mustafa/snowflake-oauth-docs
Jul 29, 2026
Merged

Add Snowflake per-user OAuth docs#2463
mustafa3rsan merged 4 commits into
mainfrom
mustafa/snowflake-oauth-docs

Conversation

@mustafa3rsan

Copy link
Copy Markdown
Contributor

No description provided.

Adds the setup/usage tutorial for Snowflake per-user OAuth (mirrors the
Databricks OAuth doc), and drops an unrequested private_key relaxation
that had crept into Config.IsValid.
Comment thread docs/cloud/ai-agents/snowflake-oauth.md Outdated
@greptile-apps

greptile-apps Bot commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Security Review

The setup guide should explicitly warn that disabling Snowflake's privileged-role OAuth safeguard is account-wide and affects integrations beyond Bruin.

Fix All in Conductor

Prompt To Fix All With AI
### Issue 1
docs/cloud/ai-agents/snowflake-oauth.md:86
**Account-wide OAuth safeguard removal**

This alternative disables privileged-role blocking for every OAuth security integration in the Snowflake account, not only `bruin_oauth`. State that account-wide impact explicitly and recommend switching to a non-privileged default role so administrators do not unintentionally broaden high-privilege OAuth access.

**How this was verified:** The documented command alters an account-level parameter rather than the Bruin security integration.

---

For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.

Reviews (1): Last reviewed commit: "Add Snowflake per-user OAuth docs, keep ..." | Re-trigger Greptile

…BLOCKED_LIST claim

Couldn't confirm this account parameter's exact behavior against
Snowflake's docs; keep only the verified BLOCKED_ROLES_LIST behavior
and the default-role workaround.
@greptile-apps

greptile-apps Bot commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Reviews (2): Last reviewed commit: "docs(snowflake-oauth): drop unverified O..." | Re-trigger Greptile

Unrelated line had been accidentally dropped from this file in an
earlier commit.
@mustafa3rsan mustafa3rsan changed the title Add Snowflake per-user OAuth docs, keep IsValid minimal Add Snowflake per-user OAuth docs Jul 28, 2026
@greptile-apps

greptile-apps Bot commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Reviews (3): Last reviewed commit: "Leave pkg/snowflake/config.go untouched ..." | Re-trigger Greptile

@mustafa3rsan
mustafa3rsan merged commit bb553ee into main Jul 29, 2026
2 checks passed
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.

2 participants