Re: [PATCH] Swift: Set Content-Length when requesting/checking Keystone tokens

[Yehuda Sadeh-Weinraub <yehuda@redhat.com>]

This looks good. Can you send a pull request to the ceph github repository?

Thanks,
Yehuda

----- Original Message -----
> From: "Herve Rousseau" <hroussea@cern.ch>
> To: ceph-devel@vger.kernel.org
> Cc: "Hervé Rousseau" <hroussea@cern.ch>
> Sent: Saturday, April 25, 2015 9:38:58 AM
> Subject: [PATCH] Swift: Set Content-Length when requesting/checking Keystone tokens
> 
> From: Hervé Rousseau <hroussea@cern.ch>
> 
> Running Keystone with WSGIChunkedRequest=On is not supported.
> 
> We have to make sure that we set the Content-Length header when getting
> an admin token and checking revoked tokens, otherwise Keystone returns
> a HTTP 411 error.
> 
> Same applies when checking revoked tickets.
> 
> Signed-off-by: Hervé Rousseau <hroussea@cern.ch>
> ---
>  src/rgw/rgw_swift.cc |    2 ++
>  1 files changed, 2 insertions(+), 0 deletions(-)
> 
> diff --git a/src/rgw/rgw_swift.cc b/src/rgw/rgw_swift.cc
> index 76f21ee..a08d944 100644
> --- a/src/rgw/rgw_swift.cc
> +++ b/src/rgw/rgw_swift.cc
> @@ -269,6 +269,7 @@ int	RGWSwift::get_keystone_admin_token(std::string&
> token)
>      std::stringstream ss;
>      jf.flush(ss);
>      token_req.set_post_data(ss.str());
> +    token_req.set_send_length(ss.str().length());
>      int ret = token_req.process("POST", token_url.c_str());
>      if (ret < 0)
>        return ret;
> @@ -296,6 +297,7 @@ int RGWSwift::check_revoked()
>      return -EINVAL;
>    url.append("v2.0/tokens/revoked");
>    req.append_header("X-Auth-Token", token);
> +  req.set_send_length(0);
>    int ret = req.process(url.c_str());
>    if (ret < 0)
>      return ret;
> --
> 1.7.1
> 
> --
> To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 
--
To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html