Changing catch to finally in PipeDuplexRequestBody.forConsume prevents long-lived streaming connections

[kohenkatz]

This change in #22 prevents the stream from staying open for long-lived connections.
For example, when handling user chat input, version 0.1.2 keeps the connection open while 0.1.3 and newer close the connection after a message is sent because the buffer is "temporarily" out of data until the user types the next message.

    fun forConsume(buffer: Buffer) {
        try {
            if (bufferedSink.isOpen) {
                bufferedSink.writeAll(buffer)
                bufferedSink.flush()
            }
-        } catch (e: Throwable) {
+        } finally {
            close()
        }
    }

I think that what should actually be here is something like this:

    fun forConsume(buffer: Buffer) {
        try {
            if (bufferedSink.isOpen) {
                bufferedSink.writeAll(buffer)
                bufferedSink.flush()
            }
+       } catch (e: Throwable) {
            close()
+           throw e;
        }
    }