Skip to content

Conversation

@sfc-gh-jwilkowski
Copy link
Contributor

…ENTIALS and OAUTH_AUTHORIZATION_CODE authenticators

Please answer these questions before submitting your pull requests. Thanks!

  1. What GitHub issue is this PR addressing? Make sure that there is an accompanying issue to your PR.

    None, tracked in SNOW-2466332

  2. Fill out the following pre-review checklist:

    • I am adding a new automated test(s) to verify correctness of my new code
    • I am adding new logging messages
    • I am adding a new telemetry message
    • I am modifying authorization mechanisms
    • I am adding new credentials
    • I am modifying OCSP code
    • I am adding a new dependency
  3. Please describe how your code solves the related issue.

    We want to simplify minimal setup for newer OAUTH based authenticators, where user needs to provide username in browser as well. Such change has already been implemented in node driver

  4. (Optional) PR for stored-proc connector:

@github-actions
Copy link

github-actions bot commented Oct 28, 2025

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@sfc-gh-jwilkowski
Copy link
Contributor Author

I have read the CLA Document and I hereby sign the CLA

…ENTIALS and OAUTH_AUTHORIZATION_CODE authenticators
@sfc-gh-jwilkowski sfc-gh-jwilkowski force-pushed the jw/SNOW-2466332-do-not-require-user-in-more-oauth-authenticators branch from 9aea1f0 to 4c73a3b Compare October 28, 2025 08:28
@sfc-gh-jwilkowski sfc-gh-jwilkowski marked this pull request as ready for review October 28, 2025 15:32
@sfc-gh-jwilkowski sfc-gh-jwilkowski requested a review from a team as a code owner October 28, 2025 15:32
@sfc-gh-turbaszek sfc-gh-turbaszek merged commit db2a10f into main Oct 29, 2025
299 of 307 checks passed
@sfc-gh-turbaszek sfc-gh-turbaszek deleted the jw/SNOW-2466332-do-not-require-user-in-more-oauth-authenticators branch October 29, 2025 13:39
@github-actions github-actions bot locked and limited conversation to collaborators Oct 29, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants