Close old client reference on configure

[damienlevin]

I think the logic around configure should be changed. There is no reason to create a default instance until configure is called.

Anyhow, this should do the trick and not break the API.

[cb372]

+1 for the idea. I've been bitten by a resource leak caused by this as well. Currently any application that uses Dispatch and calls configure is not able to shutdown, because Dispatch leaks non-daemon Netty threads.

But your implementation doesn't compile because the method is missing its braces.

[fbascheper]

@farmdawgnation AFAIK this PR was superseded by #118 , which has itself been superseded / merged by the upgrade to AHC 2.0.32.

[farmdawgnation]

Acknowledged, closing.