akka / akka-http

The Streaming-first HTTP server/module of Akka
https://doc.akka.io/docs/akka-http
Other
1.34k stars 594 forks source link

Http: client side connection abort leads to noisy warning in server #129

Open ktoso opened 8 years ago

ktoso commented 8 years ago

Issue by jrudolph Thursday Jul 16, 2015 at 13:59 GMT Originally opened as https://github.com/akka/akka/issues/18015


Also, if the connection was really aborted attempting to send 500 response is useless.

Internal server error, sending 500 response (akka.stream.StreamTcpException)

/cc @sirthias

ktoso commented 8 years ago

Comment by jrudolph Thursday Jul 16, 2015 at 14:01 GMT


On the other hand, this seems to be in line with the suggestion in https://github.com/akka/akka/issues/16842, so maybe the only thing that needs to be fixed is the message.

ktoso commented 8 years ago

Comment by Joe-Edwards Friday Jul 17, 2015 at 15:38 GMT


FWIW I've got client which always closes connections forcibly (with a reset) and my application logs are just full of these, so it'd be nice to decrease the severity or allow them to be disabled.

The message should definitely be changed though, definitely misleading.

ktoso commented 8 years ago

Comment by rkuhn Wednesday Dec 16, 2015 at 10:35 GMT


Yes, such cases are client-controlled and seem not uncommon, so it should probably be DEBUG logging.