Hi,

   sorry for the late response.

On Tue, Aug 6, 2013 at 5:10 AM, Roald van Loon <roaldvanl...@gmail.com> wrote:
> Hi ceph devs,
>
> I was working with a RGW / keystone implementation, source tree from
> github master. I stumbled against this error from the radosgw log;
>
> 2013-08-06 14:00:02.523331 7f929c011780 10 allocated request req=0x1c31640
> 2013-08-06 14:00:02.556052 7f923bfff700 10 request returned
> 2013-08-06 14:00:02.556238 7f923bfff700  0 ERROR: keystone revocation
> processing returned error r=-22
>
> That was weird (nothing after the "request returned"), because I
> checked the request using curl and it actually returned data. Then I
> found this;
>
> https://github.com/ceph/ceph/blob/master/src/rgw/rgw_swift.cc#L307
>
> Shouldn't read_data be receive_data, because the received_data is
> called by the curl_easy_perform? Or else the bufferlist in
> RGWSwift::check_revoked() will remain empty at all times.
>
> I might be missing something, so that's why I'm mailing the question
> instead of mailing the patch...

Looking at it now, this is a real issue. I recently renamed the
read_data() to receive_data() to avoid any confusion and missed these.
I'll fix it for dumpling.

Thanks,
Yehuda
--
To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to