Skip to content

feat(pam): snowflake account - #394

Open
lb-vn wants to merge 3 commits into
mainfrom
PAM-426
Open

feat(pam): snowflake account#394
lb-vn wants to merge 3 commits into
mainfrom
PAM-426

Conversation

@lb-vn

@lb-vn lb-vn commented Sep 8, 2026

Copy link
Copy Markdown
Member

Description 📣

New Snowflake account type for PAM with 3 auth methods. Supports web access and CLI access (proxy).

Type ✨

  • Bug fix
  • New feature
  • Improvement
  • Breaking change
  • Documentation

Tests 🛠️

# Here's some code block to paste some code snippets

@linear

linear Bot commented Sep 8, 2026

Copy link
Copy Markdown

PAM-426

@socket-security

socket-security Bot commented Sep 8, 2026

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Updatedgolang/​github.com/​aws/​aws-sdk-go-v2@​v1.30.3 ⏵ v1.38.171100100100100

View full report

Comment thread packages/pam/handlers/snowflake/rest.go Outdated
Comment thread packages/pam/handlers/snowflake/proxy.go
Comment thread packages/pam/handlers/snowflake/proxy.go
@veria-ai

veria-ai Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

PR overview

All previously flagged issues have been addressed. No open security concerns remain on this pull request.

Security review

No open security issues remain on this pull request.

Fixed/addressed: 3 · PR risk: 0/10

@greptile-apps

greptile-apps Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

RetriggerView in GreptileConfidence Score: 0/5

The PR is not safe to merge until credential-bearing Snowflake destinations are constrained and request and response decompression are bounded.

Findings

  1. P1 Security Account Enables Request Redirection
  2. P1 Security Decompressed Requests Are Unbounded
  3. P1 Security Result Responses Are Unbounded
  4. P2 JDBC Parameters Are Unescaped

Summary

  • Adds Snowflake login, query polling, cancellation, token renewal, result translation, and logout behavior.
  • Integrates Snowflake credentials and resource routing throughout PAM and gateway connection tests.
  • Adds focused proxy, authentication, cancellation, parsing, and uploader tests.
  • Introduces blocking outbound-request and memory-exhaustion risks that should be addressed before merge.

Comment thread packages/pam/handlers/snowflake/upstream.go
Comment thread packages/pam/handlers/snowflake/rest.go
Comment thread packages/pam/handlers/snowflake/upstream.go Outdated
Comment thread packages/pam/local/snowflake-proxy.go
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.

1 participant