Modified limits for urllib3 library

[jday1]

#171

[jkhenning]

Hi @jday1, what's the reason for this update? In general, we try not to enforce specific versions and specify the minimum required versions, in order to allow users more flexibility

[jday1]

Hi Jake,

This PR widens the allowed versions of the urllib library in keeping with ClearML's approach.

In my use case, I have libraries which I cannot upgrade cleanly as they require urllib>=2. Hence, I have raised this PR.

Cheers,
James

[jkhenning]

I'm wondering is there are any breaking changes or things that might cause issues if we use newer versions

[jday1]

I would not expect breaking changes but you could create a release candidate which users (including myself) could test out?

[jkhenning]

See the migration guide: https://urllib3.readthedocs.io/en/latest/v2-migration-guide.html
At first glance, it seems that the minimum thing to do is make sure this new constraint is only applied to python 3.7 and up (and keep the old constraint for lower versions)