Skip to content

fix: Avoid turning all stream timeouts to TcpIdleTimeoutException#32047

Open
johanandren wants to merge 1 commit into
akka:mainfrom
johanandren:wip-tcp-idle-timeout-with-less-fuzz
Open

fix: Avoid turning all stream timeouts to TcpIdleTimeoutException#32047
johanandren wants to merge 1 commit into
akka:mainfrom
johanandren:wip-tcp-idle-timeout-with-less-fuzz

Conversation

@johanandren

Copy link
Copy Markdown
Contributor

Not entirely sure this is the right thing but in akka/akka-http#4289 we noticed that StreamIdleTimeoutExceptions in the stream not originating from the TCP idle timeouts would still be turned into TcpIdleTimeoutException.

This also eliminates at least one operator in each TCP stream materialization.

Comment thread akka-stream/src/main/scala/akka/stream/impl/Timers.scala Outdated
Comment thread akka-stream/src/main/scala/akka/stream/impl/Timers.scala Outdated
@johanandren
johanandren force-pushed the wip-tcp-idle-timeout-with-less-fuzz branch from 9d40492 to d879eb8 Compare December 15, 2023 15:04
@johanandren
johanandren marked this pull request as ready for review December 15, 2023 15:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants