This looks good. Can you send a pull request to the ceph github repository?
Thanks,
Yehuda
----- Original Message -----
> From: "Herve Rousseau" <hroussea@xxxxxxx>
> To: ceph-devel@xxxxxxxxxxxxxxx
> Cc: "Hervé Rousseau" <hroussea@xxxxxxx>
> Sent: Saturday, April 25, 2015 9:38:58 AM
> Subject: [PATCH] Swift: Set Content-Length when requesting/checking Keystone tokens
>
> From: Hervé Rousseau <hroussea@xxxxxxx>
>
> 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@xxxxxxx>
> ---
> 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@xxxxxxxxxxxxxxx
> 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@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
