CONP-PCNO / CONP-PCNO/git-annex-remote-globus

suggestion: move from wget to requests

Open
#6 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
3
Forks
3
PR merge metrics
No merged PRs in 30d

Description

globus_sdk already depends on requests:
```
(git)lena:~datalad/GLOBUSannex[testremote]git
$> pip install globus_sdk
Collecting globus_sdk
Using cached https://files.pythonhosted.org/packages/be/c1/ccf8e8ebeb229887e6355e87e9cc9f07e06b1661b3c6e50311566bae9f28/globus_sdk-1.8.0-py2.py3-none-any.whl
Requirement already satisfied: six<2.0.0,>=1.10.0 in /usr/lib/python3/dist-packages (from globus_sdk) (1.12.0)
Requirement already satisfied: pyjwt[crypto]<2.0.0,>=1.5.3 in /usr/lib/python3/dist-packages (from globus_sdk) (1.7.1)
Requirement already satisfied: requests<3.0.0,>=2.9.2 in /usr/lib/python3/dist-packages (from globus_sdk) (2.21.0)
Requirement already satisfied: cryptography>=1.4 in /usr/lib/python3/dist-packages (from pyjwt[crypto]<2.0.0,>=1.5.3->globus_sdk) (2.6.1)
Installing collected packages: globus-sdk
Successfully installed globus-sdk-1.8.0
```
so imho would be better just to use requests. `wget` module seems to be not maintained since 2015 so, even though it is quite minimalistic, I expect it to eventually break with newer python versions.

Contributor guide

No contributing guide indexed for this repository

Research direction

The issue names no files or tests. Begin by locating the wget dependency and its call sites in git-annex-remote-globus, then check the existing requests dependency; done means the remote uses requests instead of wget without changing its behavior.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
cli, tooling
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.