* RGW: Refusing FastCGI request with empty CONTENT_LENGTH ? Why ?
[not found] <CAF6-1L4GdD8w=3y-0fnVHLA1RPAoKmr-EwST9_pteD_i8opkPw@mail.gmail.com>
@ 2013-04-02 13:03 ` Sylvain Munaut
2013-04-02 13:28 ` Sylvain Munaut
0 siblings, 1 reply; 5+ messages in thread
From: Sylvain Munaut @ 2013-04-02 13:03 UTC (permalink / raw)
To: ceph-devel
Hi,
In src/rgw/rgw_rest.cc you can find :
if (s->length) {
if (*s->length == '\0')
return -EINVAL;
s->content_length = atoll(s->length);
}
So that means if there is a CONTENT_LENGTH field in the environment
but it's empty, then the request is just refused.
Why ?
As it turns out, nginx does send an empty CONTENT_LENGTH params ...
which means it doesn't work at all of course.
Cheers,
Sylvain
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: RGW: Refusing FastCGI request with empty CONTENT_LENGTH ? Why ?
2013-04-02 13:03 ` RGW: Refusing FastCGI request with empty CONTENT_LENGTH ? Why ? Sylvain Munaut
@ 2013-04-02 13:28 ` Sylvain Munaut
2013-04-02 15:27 ` Sage Weil
0 siblings, 1 reply; 5+ messages in thread
From: Sylvain Munaut @ 2013-04-02 13:28 UTC (permalink / raw)
To: ceph-devel
Hi,
Replying to myself
> So that means if there is a CONTENT_LENGTH field in the environment
> but it's empty, then the request is just refused.
This is fixed in master but wasn't backported to the bobtail branch,
which is how I missed it ...
Sorry for the noise.
Cheers,
Sylvain
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: RGW: Refusing FastCGI request with empty CONTENT_LENGTH ? Why ?
2013-04-02 13:28 ` Sylvain Munaut
@ 2013-04-02 15:27 ` Sage Weil
2013-04-02 15:29 ` Mark Nelson
0 siblings, 1 reply; 5+ messages in thread
From: Sage Weil @ 2013-04-02 15:27 UTC (permalink / raw)
To: Sylvain Munaut; +Cc: ceph-devel
On Tue, 2 Apr 2013, Sylvain Munaut wrote:
> Hi,
>
> Replying to myself
>
> > So that means if there is a CONTENT_LENGTH field in the environment
> > but it's empty, then the request is just refused.
>
> This is fixed in master but wasn't backported to the bobtail branch,
> which is how I missed it ...
Which commit? We can probably backport it..
s
> Sorry for the noise.
>
> Cheers,
>
> Sylvain
> --
> 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
>
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: RGW: Refusing FastCGI request with empty CONTENT_LENGTH ? Why ?
2013-04-02 15:27 ` Sage Weil
@ 2013-04-02 15:29 ` Mark Nelson
2013-04-03 7:47 ` Sylvain Munaut
0 siblings, 1 reply; 5+ messages in thread
From: Mark Nelson @ 2013-04-02 15:29 UTC (permalink / raw)
To: Sage Weil; +Cc: Sylvain Munaut, ceph-devel
This one afaik:
https://github.com/carsonoid/ceph/commit/96896eb092c3b4e0760e56d5228ef0d604951a12
Mark
On 04/02/2013 10:27 AM, Sage Weil wrote:
> On Tue, 2 Apr 2013, Sylvain Munaut wrote:
>> Hi,
>>
>> Replying to myself
>>
>>> So that means if there is a CONTENT_LENGTH field in the environment
>>> but it's empty, then the request is just refused.
>>
>> This is fixed in master but wasn't backported to the bobtail branch,
>> which is how I missed it ...
>
> Which commit? We can probably backport it..
>
> s
>
>> Sorry for the noise.
>>
>> Cheers,
>>
>> Sylvain
>> --
>> 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
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: RGW: Refusing FastCGI request with empty CONTENT_LENGTH ? Why ?
2013-04-02 15:29 ` Mark Nelson
@ 2013-04-03 7:47 ` Sylvain Munaut
0 siblings, 0 replies; 5+ messages in thread
From: Sylvain Munaut @ 2013-04-03 7:47 UTC (permalink / raw)
To: Mark Nelson; +Cc: Sage Weil, ceph-devel
Hi,
> https://github.com/carsonoid/ceph/commit/96896eb092c3b4e0760e56d5228ef0d604951a12
Yes, exactly. The same commit is also in the official same repo with
the same commit ID.
Cheers,
Sylvain
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2013-04-03 7:47 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <CAF6-1L4GdD8w=3y-0fnVHLA1RPAoKmr-EwST9_pteD_i8opkPw@mail.gmail.com>
2013-04-02 13:03 ` RGW: Refusing FastCGI request with empty CONTENT_LENGTH ? Why ? Sylvain Munaut
2013-04-02 13:28 ` Sylvain Munaut
2013-04-02 15:27 ` Sage Weil
2013-04-02 15:29 ` Mark Nelson
2013-04-03 7:47 ` Sylvain Munaut
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.