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

Assumed AWS request URI causes problems. #6

Open
christiansegercrantz opened this issue Nov 23, 2023 · 0 comments
Open

Assumed AWS request URI causes problems. #6

christiansegercrantz opened this issue Nov 23, 2023 · 0 comments

Comments

@christiansegercrantz
Copy link

christiansegercrantz commented Nov 23, 2023

Hey,

I just found this package and it seems like a very nice one. I decided to test it out since currently a lot of my time goes to waiting for tests to pass. I think this package seems very good.

However, I ran into a problem. The package seems to assume a certain format for the result id from aws requests which, at least for my code, does not hold. I get a IndexError: list index out of range error on line this line code = request.uri.split("/results/")[1].split("/")[0] of the def _process_request_recording(request): function of snowflake_vcrpy_pytest_plugin.py (link to line). I was able to simply comment out that line and the one that depends on it to get everything running (no clue what the implications are).

The request.uristring is https://00000000-0000-0000-0000-000000000000.amazonaws.com/?accelerate

Edit: This seems to have been introduced in the latest commit to the package

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

No branches or pull requests

1 participant