Tests: Remove X-Processed-Time header

betamaxpy / betamax

A VCR imitation designed only for python-requests.

https://betamax.readthedocs.io/en/latest/

Other

565 stars 62 forks source link

Tests: Remove X-Processed-Time header #147

Closed hroncok closed 6 years ago

hroncok commented 6 years ago

httpbin.org added X-Processed-Time header that can possibly differ (and often does), so we need to get rid of it like the Date header.

Other projects noticed this as well, see libwww-perl/Net-HTTP#48

Fixes #146

hroncok commented 6 years ago

@sigmavirus24 Please let me know whether I should wait for your reviews or just go ahead and use my best judgement.

jabowery commented 6 years ago

How long does it take for the fix to show up in cpanm?

This just happened:

$ cpanm Net::HTTP
...
503 Service Unavailable at t/rt-112313.t line 80.
t/rt-112313.t .......... 
Dubious, test returned 255 (wstat 65280, 0xff00)
...
$ perl -v

This is perl 5, version 27, subversion 6 (v5.27.6) built for x86_64-linux

sigmavirus24 commented 6 years ago

@jabowery this is a python project. It's unclear what this has to do with cpanm.