Remove urllib3 dependency

[tillprochaska]

I’m not really sure if we still need this as a direct dependency. Looking at the git history, it might have been added (years ago) in order to fix an incorrect dependency constraint in a dependency (probably the Elasticsearch Python client).

Removing it doesn’t seem to cause any problems and the latest version of urllib3<2 gets installed as a transitive dependency (because it is required by elasticsearch.

[stchris]

Thanks for noticing this. I'm on board with us not pinning to a < 2 version, but I feel that if we still directly call urllib from our code we should have it as an explicit dependency and not rely on it being transitively brought in. Shall we try to bump it to the latest version?

[tillprochaska]

We're not actually loading urllib though, just setting the log level, right?

But let me check if we do need to set the log level explicitly or if we can set it for the ES client directly in some way.

[stchris]

@tillprochaska sorry, I was confused by the import. You are right, we are not using this directly so it can go!