GoogleCloudPlatform / gcs-oauth2-boto-plugin

Auth plugin allowing use the use of OAuth 2.0 credentials for Google Cloud Storage in the Boto library.
Apache License 2.0
21 stars 38 forks source link

Align setup.py and requirements.txt versions #69

Open WarheadsSE opened 5 months ago

WarheadsSE commented 5 months ago

Add missing dependencies to setup.py

Add by https://github.com/GoogleCloudPlatform/gcs-oauth2-boto-plugin/pull/62, without the entries in setup.py, pip install gsutil will result in non-functional installation.

File "/usr/local/lib/python3.9/site-packages/gcs_oauth2_boto_plugin/oauth2_client.py", line 57, in <module>
  from google_auth_httplib2 import Request as GoogleAuthRequest
ModuleNotFoundError: No module named 'google_auth_httplib2'

Update requirements.txt versions to align with setup.py for six

I noticed that these two did not agreed on the version of six, either, so I addressed that.

Closes https://github.com/GoogleCloudPlatform/gcs-oauth2-boto-plugin/issues/67

google-cla[bot] commented 5 months ago

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

WarheadsSE commented 5 months ago

CLA addressed, check needs re-run.

googlyrahman commented 5 months ago

Hi, I understand that this would be an issue with standalone plugin, but i want to understand how this is the problem with gsutil installation, as gsutil setup.py (https://github.com/GoogleCloudPlatform/gsutil/blob/master/setup.py#L55) consists of these dependencies. I've also tried doing a fresh installation on a fresh VM, and it works perfectly fine, as gsutil setup.py has those dependency.

googlyrahman commented 5 months ago

Thanks for bringing this to our attention, We were also discussing this internally, and decided to fix it.

WarheadsSE commented 5 months ago

Hi, I understand that this would be an issue with standalone plugin, but i want to understand how this is the problem with gsutil installation, as gsutil setup.py (https://github.com/GoogleCloudPlatform/gsutil/blob/master/setup.py#L55) consists of these dependencies.

Put simply, you are correct. As of 2 weeks ago, master of gsutil did indeed include that dependency. However, anyone who installed a version prior to that point in time, would have a floating dependency on this package, which would not have declared a functional requirement.

stanhu commented 5 months ago

For example, if you had run pip install gsutil==5.27 prior to the release of gcs-oauth2-boto-plugin v3.2, then you would run into this issue.

WarheadsSE commented 5 months ago

@googlyrahman @shilpi23pandey I have left this open, for the time being, in regards to change in requirements.txt. Consider it a reminder to address that.

70 effectively took the changes from here, and #68, but placed them under different lines, and with your internal team as the author(s).

shilpi23pandey commented 5 months ago

@googlyrahman @shilpi23pandey I have left this open, for the time being, in regards to change in requirements.txt. Consider it a reminder to address that.

70 effectively took the changes from here, and #68, but placed them under different lines, and with your internal team as the author(s).

Hi @WarheadsSE This issue was blocking a critical gsutil release which was raised internally so we had to take action ASAP. For the sake of time, we decided to fix it ourselves.

That said, thanks for the PR and logging the issue.