[tarantool-patches] Re: [PATCH v2 1/1] http: fix httpc auto-managed headers

Vladimir Davydov vdavydov.dev at gmail.com
Tue Mar 12 11:54:24 MSK 2019


On Mon, Mar 11, 2019 at 06:57:54PM +0300, Kirill Shcherbatov wrote:
> The http library intelligently sets the headers "Accept",
> "Connection", "Keep-Alive".
> However, when the user explicitly specified them in the header
> options section of the call argument, they could be written to
> the HTTP request twice.
> We postponed the auto headers setup before https_exececute call.
> Now they are set only if they were not set by the user.
> 
> Closes #3955
> 
> https://github.com/tarantool/tarantool/tree/kshch/gh-3955-httpc-auto-managed-headers-redefine
> https://github.com/tarantool/tarantool/issues/3955
> ---
>  src/httpc.c                       | 52 +++++++++++++++++++++----------
>  src/httpc.h                       | 23 +++++++++++++-
>  src/lua/httpc.c                   |  6 +---
>  test/app-tap/http_client.test.lua | 36 ++++++++++++++++++++-
>  4 files changed, 94 insertions(+), 23 deletions(-)

Pushed to 2.1 and 1.10.



More information about the Tarantool-patches mailing list