Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

t/local/*.t may hang #278

Open
eserte opened this issue Aug 2, 2019 · 7 comments
Open

t/local/*.t may hang #278

eserte opened this issue Aug 2, 2019 · 7 comments
Labels

Comments

@eserte
Copy link

eserte commented Aug 2, 2019

Recently the WWW::Mechanize test suite started to hang on some my smokers (various OS, various perl versions). The hangs always happened in some of the t/local/*.t tests: mostly referer.t or back.t, sometimes also encoding.t. Before the hanging test it seems that there was always a "Connection refused" error. Sample reports on CPAN Testers:

@genio
Copy link
Member

genio commented Aug 2, 2019

Thanks, @eserte This is due to #272 and #101

HTTP::Daemon was updated to be IPv6 compatible. The tests herein need to be fixed to accommodate the possibility of an IPv6 socket being opened.

@skaji
Copy link
Member

skaji commented Aug 4, 2019

@eserte Can you try #280 ?

@skaji
Copy link
Member

skaji commented Aug 5, 2019

@eserte If #280 does not work for you,
could you also run this script?
https://gist.github.com/skaji/d0a71d9f02576a2f529798e1d8f6a98d

See also libwww-perl/HTTP-Daemon#33

@eserte
Copy link
Author

eserte commented Aug 8, 2019

The t/local* tests were always hanging with the system perl on my centos7 smoker, and are still hanging with #280. On two other configurations there were no problems.

The gist already dies on the $Net::HTTP::SOCKET_CLASS eq 'IO::Socket::IP' or die; line.

@skaji
Copy link
Member

skaji commented Aug 8, 2019

@eserte
Both #280 and the gist assume that you have HTTP::Daemon 6.05+ (and its dependency IO::Socket::IP).

So could you try again #280 and the gist after installing HTTP::Daemon 6.05+?

@eserte
Copy link
Author

eserte commented Aug 21, 2019

I'd say: ignore the problems on the centos7 machine with the system perl --- WWW::Mechanize's test suite was always hanging here, so maybe it's a different problem (and doing stuff on this configuration is a little bit difficult for me, because of policy decisions I do not install anything here, just run CPAN.pm with the test action).

For other systems it seems that the PR code solves the problems. So a 👍 from me for the PR.

@petdance petdance added the tests label Sep 14, 2020
@jkeenan
Copy link

jkeenan commented Jul 3, 2023

I'd say: ignore the problems on the centos7 machine with the system perl --- WWW::Mechanize's test suite was always hanging here, so maybe it's a different problem (and doing stuff on this configuration is a little bit difficult for me, because of policy decisions I do not install anything here, just run CPAN.pm with the test action).

For other systems it seems that the PR code solves the problems. So a +1 from me for the PR.

Can you tell us whether this problem has cleared up now that there have been new CPAN releases of WWW-Mechanize and HTTP-Daemon?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants