Skip to content

Snow 2183023 python wif in driver sdk boto3 replacement with explicit http requests #2391

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 57 commits into
base: main
Choose a base branch
from
Open

Snow 2183023 python wif in driver sdk boto3 replacement with explicit http requests #2391

wants to merge 57 commits into from
+1,084 −342

Conversation

sfc-gh-fpawlowski
Copy link
Contributor

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.

    Fixes #NNNN

  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.

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

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

@sfc-gh-fpawlowski sfc-gh-fpawlowski self-assigned this Jul 7, 2025
@sfc-gh-fpawlowski sfc-gh-fpawlowski added the DO_NOT_PORT_CHANGES_TO_SP Add this label when changes in this PR do not need to be port to SP connector label Jul 7, 2025
@sfc-gh-fpawlowski
SNOW-2183023: SEssions map access fixed
f872fd2
@sfc-gh-fpawlowski
SNOW-2183023: use pooling
95ace1f
@sfc-gh-fpawlowski
SNOW-2183023: refactored
53087d4
@sfc-gh-fpawlowski
SNOW-2183023: fixed fake env
23e338b
@sfc-gh-fpawlowski
Revert "SNOW-2183023: refactored"
3faa942 
This reverts commit 53087d4.
@sfc-gh-fpawlowski
Reapply "SNOW-2183023: refactored"
43f7868 
This reverts commit 3faa942.
@sfc-gh-fpawlowski
SNOW-2183023: fixed csp_helper
2241969
@sfc-gh-fpawlowski
Revert "Reapply "SNOW-2183023: refactored""
4f0785c 
This reverts commit 43f7868.
@sfc-gh-fpawlowski
Reapply "Reapply "SNOW-2183023: refactored""
e603c1c 
This reverts commit 4f0785c.
@sfc-gh-fpawlowski
SNOW-2183023: fixed csp_helper and added boto as dev-dep
fde7556
@sfc-gh-fpawlowski
SNOW-2183023: added arn fetch
023f0ae
@sfc-gh-fpawlowski
Revert "Reapply "Reapply "SNOW-2183023: refactored"""
3fef4e7 
This reverts commit e603c1c.
@sfc-gh-fpawlowski
Reapply "Reapply "Reapply "SNOW-2183023: refactored"""
2e23fe3 
This reverts commit 3fef4e7.
@sfc-gh-fpawlowski
SNOW-2183023: fixed missing prepare args
59a3373
@sfc-gh-fpawlowski
SNOW-2183023: fixed missing prepare args
093b5b5
@sfc-gh-fpawlowski
SNOW-2183023: fixed patching
219e8c6
@sfc-gh-fpawlowski
SNOW-2183023: fixed tests
b486239
@sfc-gh-fpawlowski
SNOW-2183023: fixed tests
af2f4a3
@sfc-gh-fpawlowski
SNOW-2183023: fixed tests
88ccb83
@sfc-gh-fpawlowski sfc-gh-fpawlowski marked this pull request as ready for review July 8, 2025 08:03
@sfc-gh-fpawlowski
SNOW-2183023: fixed user id
a3faefd
@sfc-gh-fpawlowski
SNOW-2183023: fixed mock
a2c0295
@sfc-gh-fpawlowski
wif credential working
e176195
@sfc-gh-fpawlowski
get region working
1efb814
@sfc-gh-fpawlowski
signing not yet working
ec76ade
@sfc-gh-fpawlowski
that worked
12cf2cd
@sfc-gh-fpawlowski
boto3 removed
0151c7d
@sfc-gh-fpawlowski
botocore removed. cleanup
39b3267
@sfc-gh-fpawlowski
botocore removed. cleanup
110a63b
@sfc-gh-fpawlowski
cleanup to own Credential class
bd58179
@sfc-gh-fpawlowski
refactored files
564ac06
@sfc-gh-fpawlowski
arn from env vars or only region
8df9bdf
@sfc-gh-fpawlowski
tests working
04a11d8
@sfc-gh-fpawlowski
botocore compatibility tests
f018af5
@sfc-gh-fpawlowski
finished boto comp
b5e3554
@sfc-gh-fpawlowski
SNOW-2183023: removed session manager
6ab7a50
@sfc-gh-fpawlowski
Refactored without sessionManager - split into 2 PRs
e85216a
@sfc-gh-fpawlowski
SNOW-2183023: slight improved tests
bbf5052
@sfc-gh-fpawlowski
SNOW-2183023: TEMP commit metadata service initial intro
35e23cf
@sfc-gh-fpawlowski
SNOW-2183023: All tests passing with many aws envs
0b8c7a6
@sfc-gh-fpawlowski
SNOW-2183023: all tests pass and only http is mocked
f1fd5fa
@sfc-gh-fpawlowski
Merge branch 'main' into SNOW-2183023-Python-WIF-in-Driver-SDK-boto3-…
855ff4f 
…replacement-with-explicit-http-requests
@sfc-gh-fpawlowski
SNOW-2183023: base fix of constants approach
2bc2825
@sfc-gh-fpawlowski
SNOW-2183023: base fix of constants approach
ea742d6
@sfc-gh-fpawlowski
SNOW-2183023: this breaks code a lot - wrong order of envs cleanup
0bee4cd 
compare with 2 commits ago
@sfc-gh-fpawlowski
SNOW-2183023: breaks code less - wrong order of envs cleanup
e3a00ab 
compare with 2 commits ago
@sfc-gh-fpawlowski
SNOW-2183023: fixed http traffix
fbbc223
@sfc-gh-fpawlowski
SNOW-2183023: fixed http traffix
fab3af9
@sfc-gh-fpawlowski
SNOW-2183023: comments cleanup
f9b8918
@sfc-gh-fpawlowski
SNOW-2183023: http session unmanaged requests
935efd1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sfc-gh-pmansour sfc-gh-pmansour Awaiting requested review from sfc-gh-pmansour

At least 1 approving review is required to merge this pull request.

Assignees

@sfc-gh-fpawlowski sfc-gh-fpawlowski

Labels
DO_NOT_PORT_CHANGES_TO_SP Add this label when changes in this PR do not need to be port to SP connector
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@sfc-gh-fpawlowski