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

decorate UnexpectedConnectionClosureException with IOException #228

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

jtjeferreira
Copy link
Contributor

Decorate akka-http exceptions with exceptionds understood by the AWS SDK so that they are automatically retried by the default retry policy. This was inspired in NettyUtils.decorateException

Since UnexpectedConnectionClosureException is private we need to use a hack...

@jtjeferreira
Copy link
Contributor Author

As one can see from the build logs of 8c7a211, the test fails as expected

[info]   Cause: akka.http.impl.engine.client.OutgoingConnectionBlueprint$UnexpectedConnectionClosureException: The http server closed the connection unexpectedly before delivering responses for 1 outstanding requests

João Ferreira and others added 2 commits October 23, 2023 14:01
…han 10s to boot

11:04:01.130 [pool-1-thread-1] INFO  t.a.1.0 - Container adobe/s3mock:2.13.0 is starting: 85d5881648cb02a95ecc3d406cf0b70ca7a6638c42a0d5e1f800c783479ee14e
11:04:11.530 [pool-1-thread-1] INFO  t.a.1.0 - Container adobe/s3mock:2.13.0 started in PT10.436S
@jtjeferreira
Copy link
Contributor Author

In the logs of c6c8228 I saw a new exception akka.http.impl.engine.client.pool.SlotState$BusyState$$anon$1: Connection was shutdown. This seems very similar to akka/akka-http#2349 (which supposedly was already fixed) but seems to still be happening in akka-http 10.5.2

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

Successfully merging this pull request may close these issues.

1 participant