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

luminous: rgw: multisite: object name should be urlencoded when we put it into ES #24424

Merged
merged 1 commit into from Oct 5, 2018

Conversation

Projects
None yet
5 participants
@theanalyst
Member

theanalyst commented Oct 4, 2018

rgw: url_encode key name and instance in es sync module
Some objects whose name contains space or other special chars
can't be synced to ES correctly. we need to do url_encode when
we send a HTTP request to ES.

Fixes: tracker.ceph.com/issues/23216

Signed-off-by: Chang Liu <liuchang0812@gmail.com>
(cherry picked from commit 13978bb)

@theanalyst theanalyst requested review from yehudasa and cbodley Oct 4, 2018

@smithfarm smithfarm added this to the luminous milestone Oct 4, 2018

@yuriw

This comment has been minimized.

Contributor

yuriw commented Oct 4, 2018

@cbodley

cbodley approved these changes Oct 5, 2018

@yuriw yuriw merged commit 593d3a4 into ceph:luminous Oct 5, 2018

4 checks passed

Docs: build check OK - docs built
Details
Signed-off-by all commits in this PR are signed
Details
Unmodified Submodules submodules for project are unmodified
Details
make check make check succeeded
Details

@smithfarm smithfarm changed the title from luminous: multi-site: object name should be urlencoded when we put it into ES to luminous: rgw: multisite: object name should be urlencoded when we put it into ES Oct 26, 2018

@smithfarm smithfarm changed the title from luminous: rgw: multisite: object name should be urlencoded when we put it into ES to luminous: rgw: multisite: object name should be urlencoded when we put it into ES Oct 26, 2018

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