Skip to content
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.

Sign up for GitHub

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

Jump to bottom

kraken: rgw: multisite: rest api fails to decode large period on 'period commit' #16161

Merged
merged 6 commits into from
Jul 19, 2017
Merged

kraken: rgw: multisite: rest api fails to decode large period on 'period commit' #16161

merged 6 commits into from
Jul 19, 2017

Commits on Jul 6, 2017

  1. rgw: make a configurable size for requests with xml params 

    We currently read all user input for a few apis accepting xml, avoid
doing this and error out early, most s3 apis limit to about 1000 xml
entries, make this configurable via a new config param,
`rgw_max_put_param_size` defaulting to 1MB. Also modified
`rgw_rest_read_all_input` with an additional param to not allow for
chunked uploads, though we error out in a better way by responding with
content length required rather than S3's 501 if chunked is set.

This also adds the same behavior in RGWPutCORS reusing
`rgw_rest_read_all_input`, and using a data_deleter to manage the char*

Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>

rgw: rgw_rest_s3: make PUT CORS accept only a configured max input

Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>
(cherry picked from commit 35375b7)
    @theanalyst @smithfarm
    theanalyst authored and smithfarm committed Jul 6, 2017
    Configuration menu
    Copy the full SHA
    dea9a69 View commit details
    Browse the repository at this point in the history

  2. rgw: allow larger payload for period commit 

    testing with 3 zonegroups and 3 zones each, the period json grew larger
than 4k and caused decode failures on period commit

updated to use the new config variable rgw_max_put_param_size

Fixes: http://tracker.ceph.com/issues/19505

Signed-off-by: Casey Bodley <cbodley@redhat.com>
(cherry picked from commit 7f2871f)
    @cbodley @smithfarm
    cbodley authored and smithfarm committed Jul 6, 2017
    Configuration menu
    Copy the full SHA
    f23df08 View commit details
    Browse the repository at this point in the history

  3. rgw: use zonegroup's master zone endpoints for bucket redirect 

    if no zonegroup endpoints are set, fall back to master zone endpoints

Fixes: http://tracker.ceph.com/issues/19488

Signed-off-by: Casey Bodley <cbodley@redhat.com>
(cherry picked from commit 92f63c6)
    @cbodley @smithfarm
    cbodley authored and smithfarm committed Jul 6, 2017
    Configuration menu
    Copy the full SHA
    693d880 View commit details
    Browse the repository at this point in the history

  4. rgw: fix for zonegroup redirect url 

    local dest_url variable was shadowing the one in the enclosing scope, so
the changes were not applied and no Location header was written on redirect

Fixes: http://tracker.ceph.com/issues/19488

Signed-off-by: Casey Bodley <cbodley@redhat.com>
(cherry picked from commit 542e188)
    @cbodley @smithfarm
    cbodley authored and smithfarm committed Jul 6, 2017
    Configuration menu
    Copy the full SHA
    4e4cd54 View commit details
    Browse the repository at this point in the history

  5. radosgw-admin: warn that 'realm rename' does not update other clusters 

    Fixes: http://tracker.ceph.com/issues/19746

Signed-off-by: Casey Bodley <cbodley@redhat.com>
(cherry picked from commit 8a459c7)
    @cbodley @smithfarm
    cbodley authored and smithfarm committed Jul 6, 2017
    Configuration menu
    Copy the full SHA
    9651ed9 View commit details
    Browse the repository at this point in the history

  6. rgw: only log metadata on metadata master zone 

    Fixes: http://tracker.ceph.com/issues/20244

Signed-off-by: Casey Bodley <cbodley@redhat.com>
(cherry picked from commit b8272f3)
    @cbodley @smithfarm
    cbodley authored and smithfarm committed Jul 6, 2017
    Configuration menu
    Copy the full SHA
    d52aeec View commit details
    Browse the repository at this point in the history