On Sun, Jan 19, 2025 at 12:09:47AM +0100, Santiago Vila wrote:
> This is very strange.
> 
> Now pytest-httpbin builds ok but vcr.py still fails to build.
> (on the systems where it used to fail for me)
> 
> So, I'm going to consider this as a bug in vcr.py only.
> 
> If somebody wants to debug this, please contact me privately.
> 
> If nobody helps, I will apply the "bureaucratic procedure": forward
> upstream and disable the failing tests via team upload.

I've posted my debugging notes in
https://github.com/kevin1024/vcrpy/issues/905#issuecomment-2619147042.
The short answer is that I don't see how pytest-httpbin (or packages
using it) can work reliably when test requests with "Transfer-Encoding:
chunked" are involved: they will pass or fail depending on whether the
test manages to send the full request body before httpbin closes the
connection with a 501 response (and they'll only pass if they aren't
interested in testing the response status).  Although some new unit
tests were introduced in vcr.py 6.0.2 that also run into this, the
affected integration tests are older, and the fundamental problem seems
to have been there for years.  Bisecting will probably be unreliable.
It doesn't seem to have anything to do with recent changes in either
urllib3 or Python, although of course it's possible that some innocuous
changes perturbed timing a bit.

However, since this is specific to testing with httpbin instances that
aren't running via gunicorn (in particular pytest-httpbin), treating the
tests as flaky and disabling them in the Debian packaging seems
reasonable to me for now.

I think I've got as far as I can with debugging this.  Santiago, thanks
for the test VM; you can shut it down now.

-- 
Colin Watson (he/him)                              [cjwat...@debian.org]

Reply via email to