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

materialize-snowflake: remove password authentication as an option #2390

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

mdibaiee
Copy link
Member

@mdibaiee mdibaiee commented Feb 12, 2025

Description:

see https://www.snowflake.com/en/blog/blocking-single-factor-password-authentification/

Workflow steps:

(How does one use this feature, and how has it changed)

Documentation links affected:

(list any documentation links that you created, or existing ones that you've identified as needing updates, along with a brief description)

Notes for reviewers:

(anything that might help someone review this PR)


This change is Reviewable

@mdibaiee mdibaiee marked this pull request as ready for review February 12, 2025 17:56
@willdonnelly
Copy link
Member

Three questions:

  1. Do we actually validate endpoint configs against the schema anywhere that might be an issue here?
  2. If the user tries to make a unrelated change via the UI, like adding/disabling a binding or triggering a backfill, will they be forced to also change their authentication mode or will the existing endpoint spec just be passed through unmodified?
  3. I'm pretty sure that an unrelated change to the endpoint spec will definitely force them to go reconfigure their authentication mode. Is that desirable now given that there are still 9 months to go before username/password auth is disabled?

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