jclouds-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Ignasi Barrera <notificati...@github.com>
Subject [jclouds] Allow to globally configure the 'Connection: close' header (#862)
Date Fri, 02 Oct 2015 13:20:00 GMT
Some providers do not properly support persistent connections. See: 
https://github.com/docker/docker/issues/12845
https://github.com/docker/compose/pull/1374

This pull request introduces the to configure `jclouds.connection-close-header`, to automatically
add the `Connection: close` header to all requests. This is particularly useful when using
drivers that reuse connections by default (such as the [OkHttp driver](https://github.com/square/okhttp/wiki/Connections)),
but still allows to configure the header explicitly in concrete methods of concrete APIs where
needed.

/cc @kwart
You can view, comment on, or merge this pull request online at:

  https://github.com/jclouds/jclouds/pull/862

-- Commit Summary --

  * Allow to globally configure the 'Connection: close' header

-- File Changes --

    M core/src/main/java/org/jclouds/Constants.java (6)
    M core/src/main/java/org/jclouds/apis/internal/BaseApiMetadata.java (3)
    A core/src/main/java/org/jclouds/http/filters/ConnectionCloseHeader.java (33)
    M core/src/main/java/org/jclouds/rest/internal/RestAnnotationProcessor.java (10)
    A core/src/test/java/org/jclouds/http/filters/ConnectionCloseHeaderTest.java (38)
    M core/src/test/java/org/jclouds/rest/internal/RestAnnotationProcessorTest.java (29)

-- Patch Links --

https://github.com/jclouds/jclouds/pull/862.patch
https://github.com/jclouds/jclouds/pull/862.diff

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/862

Mime
View raw message