search

erlyaws / yaws

Yaws webserver
https://erlyaws.github.io
BSD 3-Clause "New" or "Revised" License
1.28k stars 267 forks source link

Unhandled reply fr. do_recv() {error,etimedout} #355

Closed leoliu closed 5 years ago

leoliu commented 5 years ago

I am seeing this printed in the console:

=ERROR REPORT==== 25-Dec-2018::14:44:43.781628 ===
Unhandled reply fr. do_recv() {error,etimedout}

which is originated from yaws:http_recv_request/0. Is {error, timeout} supposed to be {error,etimedout}? Another issue is fr. is an odd abbreviation.

vinoski commented 5 years ago

{error, timeout} is valid, and so I've left it alone in 517ca65, but I've added handling for {error, etimedout} too, and I've always wanted to fix that log message and so I've done that as well.

leoliu commented 5 years ago

@vinoski The change looks good. Thanks.

vinoski commented 5 years ago

Rebased 517ca65 to master.

leoliu commented 5 years ago

@vinoski I can't find the commit on master. Something wrong?

vinoski commented 5 years ago

Yes, apparently there was a network glitch while I pushed it. It's there now, commit 517ca65.

leoliu commented 5 years ago

Thanks. Works now.