Hi, sorry for the late response. On Tue, Aug 6, 2013 at 5:10 AM, Roald van Loon <roaldvanloon@xxxxxxxxx> 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 majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
