Double Disconnect

[benlangfeld]

Ok, so I found the issue causing this bug: http://github.com/metajack/strophejs/issues#issue/2

If more than one connection is allowed, the terminate/unavailable stanza is sent on a new connection, which gets a terminate response. There is still another open connection, however, so instead of the final part of closing a connection being performed:

    // handle graceful disconnect
    if (this.disconnecting && this._requests.length === 0) {
        this.deleteTimedHandler(this._disconnectTimeout);
        this._disconnectTimeout = null;
        this._doDisconnect();
        return;
    }

We actually consider the terminate response to be a connection error. In order to handle this correctly, we need to cease considering this as an error, by testing if a disconnect is in progress.

[metajack]

Thanks for this fix. I added a comment and merge your changes.