Skip to content

Conversation

sfc-gh-pcyrek
Copy link
Contributor

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

SNOW-2235955: adding MFA test in Python

  1. 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
  2. Please describe how your code solves the related issue.

    Please write a short description of how your code change solves the related issue.

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

@sfc-gh-pcyrek sfc-gh-pcyrek added NO-CHANGELOG-UPDATES This pull request does not need to update CHANGELOG.md DO_NOT_PORT_CHANGES_TO_SP Add this label when changes in this PR do not need to be port to SP connector labels Aug 11, 2025
Copy link
Contributor

@sfc-gh-akolodziejczyk sfc-gh-akolodziejczyk left a comment

Choose a reason for hiding this comment

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

LGTM

@sfc-gh-pcyrek sfc-gh-pcyrek merged commit 5c5b737 into main Aug 18, 2025
94 of 109 checks passed
@sfc-gh-pcyrek sfc-gh-pcyrek deleted the SNOW-2235955-adding-mfa-tests-in-python branch August 18, 2025 08:47
@github-actions github-actions bot locked and limited conversation to collaborators Aug 18, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
DO_NOT_PORT_CHANGES_TO_SP Add this label when changes in this PR do not need to be port to SP connector NO-CHANGELOG-UPDATES This pull request does not need to update CHANGELOG.md
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants