Add yaml-based tests for host header and sni checking

[cdent]

The addition of server_hostname to the httpclient PoolManager, without sufficient testing, has revealed some issues:

• The minimum urllib3 required is too low. server_hostname was introduced in 1.24.x
• However, there is a bug [1] in PoolManager when mixing schemes in the same pool manager. This is being fixed so the minimum urllib3 will need to be higher still.

Tests are added here, and the minimum value for urllib3 will be set when a release is made.

Some of the tests are "live" meaning they require network, and can be skipped via the live test fixture if the GABBI_SKIP_NETWORK env variable is set to "true".

[1] https://github.com/urllib3/urllib3/issues/2534

Fixes #307 Fixes #309

[cdent]

Note that these tests are expected to fail until a new version of urllib3 becomes available after https://github.com/urllib3/urllib3/pull/2537 merges.

[scottwallacesh]

It appears https://github.com/urllib3/urllib3/issues/2534 is closed now. Will the tests now pass?

[cdent]

The issue is that there has not yet been a new release to pypi. I've inquired a few different times (and again today).

[cdent]

For historical reference: There was a 2.5.0 release, but I had to yank it as it failed to work properly with urllib3. The fix to urllib3 has not been released.

[scottwallacesh]

There was a new urllib3 release (v1.26.9) on the 16th! Your message must have prodded them.

[cdent]

There was a new urllib3 release (v1.26.9) on the 16th!

Yup, dealing with this is on my list for today. We should see a 2.5.1 release in the next few hours.

[cdent]

As a status update: I'm having some very bizarre test failures in py39-pytest, but not py39, which shouldn't be possible, but clearly they are happening. Still digging.

[cdent]

ooph, finally: https://pypi.org/project/gabbi/