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.

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

Missing dependency message does not appear when it should #441

Open
d33bs opened this issue Jun 13, 2024 · 2 comments
Open

Missing dependency message does not appear when it should #441

d33bs opened this issue Jun 13, 2024 · 2 comments

Comments

@d33bs
Copy link

d33bs commented Jun 13, 2024

Thanks for the great work on cloudpathlib! I noticed a bug in the documentation for using no_sign_request=True found here. When I attempt to use the example code, an error shows: NameError: name 'Session' is not defined. Here's an example reproducing the error: https://gist.github.com/d33bs/eb19f864baae89757e0e0969611cf1af
@d33bs d33bs mentioned this issue Jun 13, 2024
Closed
@pjbull
Copy link
Member

pjbull commented Jun 13, 2024

Do you have boto3 and botocore installed as well and can you provide the version? They should get installed when you do pip install "cloudpathlib[s3]".

You should get a more helpful error if you are missing the SDK for a particular cloud provider, but based on your example, I think that is what may be happening.

@d33bs
Copy link
Author

d33bs commented Jun 13, 2024

Thanks @pjbull ! This ended up being the issue - after reinstalling with "cloudpathlib[s3]" things worked! Thanks for the prompt reply to this.

@pjbull pjbull changed the title no_sign_request=True examples in documentation appear to emit errors Missing dependency message does not appear when it should Jun 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@pjbull @d33bs