Thanks I'll take another look

I've managed to reproduce by using the 2.1.2 distribution directly. I'm unsure why my customised build using the 2.1.2 tag isn't working though... I shall look further as it is important to be able to prove that any fix on 2.1.x actually fixes the issue.

I think I've found the problem - it has something to do with the re-factorings that were done around the upstream handler Take this bit of code of the upstream handler:

Iteratee.flatten(
  NettyPromise(future)
    .map(_ => if (ctx.getChannel.isConnected()) Cont(step) else Done((), Input.Empty)))

map is called only on success of course. If there's failure i.e. the connection dies (which it will for this scenario) then the Iteratee will not be completed with a Done.

There must be some difference in the code we package in order to mask a race condition. I shall attend to a fix, though it may be tricky to reproduce!

Here's a patch for review on the 2.1.x branch. I'll also create a PR and attach it to this issue for the master branch - the changes will be different for master.

LGTM

play2-master-PRs #364 SUCCESS
This pull request looks good

Just as a footnote to this issue, I was finally able to package a 2.1.x release and reproduce the issue. I then applied my 2.1.x fix, packaged again and re-tested. The issue appeared to be resolved.