New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

rgw: DragonDisk fails to create directories via S3: MissingContentLength #5118

Merged
3 commits merged into from Jul 31, 2015

Conversation

Projects
None yet
4 participants
@theanalyst
Member

theanalyst commented Jul 1, 2015

robbat2 and others added some commits Apr 22, 2015

rgw: improve content-length env var handling
The FastCGI specification, section 6.3 on Authorizers, describes a case
where HTTP_CONTENT_LENGTH will be set in the environment and
CONTENT_LENGTH will NOT be set.

Further documention in the code.

Fixes: #11419
Signed-off-by: Robin H. Johnson <robbat2@gentoo.org>
(cherry picked from commit 3e38eab)
rgw: make compatability deconfliction optional.
Per request from Yehuda, the deconfliction for having both
HTTP_CONTENT_LENGTH and CONTENT_LENGTH set is now optional, and
controlled by new configuration boolean, which defaults to false.
rgw content length compat

X-URL: #4436 (comment)
Signed-off-by: Robin H. Johnson <robbat2@gentoo.org>
(cherry picked from commit 79d17af)
rgw: simplify content length handling
Signed-off-by: Yehuda Sadeh <yehuda@redhat.com>
(cherry picked from commit e97fd50)

@theanalyst theanalyst self-assigned this Jul 1, 2015

@theanalyst theanalyst added this to the hammer milestone Jul 1, 2015

ghost pushed a commit that referenced this pull request Jul 15, 2015

Merge pull request #5118: DragonDisk fails to create directories via …
…S3: MissingContentLength

Reviewed-by: Loic Dachary <ldachary@redhat.com>

ghost pushed a commit that referenced this pull request Jul 15, 2015

Merge pull request #5118: DragonDisk fails to create directories via …
…S3: MissingContentLength

Reviewed-by: Loic Dachary <ldachary@redhat.com>

ghost pushed a commit that referenced this pull request Jul 16, 2015

Merge pull request #5118: DragonDisk fails to create directories via …
…S3: MissingContentLength

Reviewed-by: Loic Dachary <ldachary@redhat.com>

ghost pushed a commit that referenced this pull request Jul 19, 2015

Merge pull request #5118: DragonDisk fails to create directories via …
…S3: MissingContentLength

Reviewed-by: Loic Dachary <ldachary@redhat.com>

ghost pushed a commit that referenced this pull request Jul 19, 2015

Merge pull request #5118: DragonDisk fails to create directories via …
…S3: MissingContentLength

Reviewed-by: Loic Dachary <ldachary@redhat.com>

ghost pushed a commit that referenced this pull request Jul 19, 2015

Merge pull request #5118: DragonDisk fails to create directories via …
…S3: MissingContentLength

Reviewed-by: Loic Dachary <ldachary@redhat.com>

ghost pushed a commit that referenced this pull request Jul 19, 2015

Merge pull request #5118: DragonDisk fails to create directories via …
…S3: MissingContentLength

Reviewed-by: Loic Dachary <ldachary@redhat.com>

ghost pushed a commit that referenced this pull request Jul 19, 2015

Merge pull request #5118: DragonDisk fails to create directories via …
…S3: MissingContentLength

Reviewed-by: Loic Dachary <ldachary@redhat.com>

@theanalyst theanalyst added rgw and removed core labels Jul 28, 2015

@ghost

This comment has been minimized.

ghost commented Jul 28, 2015

@yehudasa does this hammer backport looks good to merge ? It went through a rgw suite ( http://tracker.ceph.com/issues/11990#rgw ), only one job failed because of a radosgw-agent failure specific to Ubuntu 12.04 (but succeded on 14.04).

ghost pushed a commit that referenced this pull request Jul 29, 2015

ghost pushed a commit that referenced this pull request Jul 31, 2015

Loic Dachary
Merge pull request #5118 from theanalyst/wip-12042-hammer
DragonDisk fails to create directories via S3: MissingContentLength

Reviewed-by: Yehuda Sadeh <ysadehwe@redhat.com>

@ghost ghost merged commit 07fa83a into ceph:hammer Jul 31, 2015

@theanalyst theanalyst deleted the theanalyst:wip-12042-hammer branch Aug 1, 2015

@ghost ghost changed the title from DragonDisk fails to create directories via S3: MissingContentLength to rgw: DragonDisk fails to create directories via S3: MissingContentLength Aug 4, 2015

This issue was closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment