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: Ensure that swift keys don't include backslashes #5716

Merged
1 commit merged into from Sep 3, 2015

Conversation

Projects
None yet
3 participants
@ghost

ghost commented Aug 30, 2015

rgw: avoid using slashes for generated secret keys
Just use plain alphanumeric characterset.

Fixes: #7647

Signed-off-by: Yehuda Sadeh <yehuda@redhat.com>
(cherry picked from commit 3f04a61)

@ghost ghost self-assigned this Aug 30, 2015

@ghost ghost added this to the hammer milestone Aug 30, 2015

@ghost ghost added bug fix rgw labels Aug 30, 2015

ghost pushed a commit that referenced this pull request Aug 30, 2015

Merge pull request #5716: Ensure that swift keys don't include backsl…
…ashes

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

ghost pushed a commit that referenced this pull request Aug 30, 2015

Merge pull request #5716: Ensure that swift keys don't include backsl…
…ashes

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

ghost pushed a commit that referenced this pull request Aug 30, 2015

Merge pull request #5716: Ensure that swift keys don't include backsl…
…ashes

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

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

Merge pull request #5716: Ensure that swift keys don't include backsl…
…ashes

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

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

Merge pull request #5716: Ensure that swift keys don't include backsl…
…ashes

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

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

Merge pull request #5716: Ensure that swift keys don't include backsl…
…ashes

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

ghost pushed a commit that referenced this pull request Sep 1, 2015

Merge pull request #5716: Ensure that swift keys don't include backsl…
…ashes

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

ghost pushed a commit that referenced this pull request Sep 2, 2015

Merge pull request #5716: Ensure that swift keys don't include backsl…
…ashes

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

This comment has been minimized.

ghost commented Sep 2, 2015

@oritwas does this backport look good to merge ? It passed a run of the hammer rgw suite ( see http://pulpito.ceph.com/loic-2015-09-01_17:25:32-rgw-hammer-backports---basic-multi/ ).

@oritwas

This comment has been minimized.

Contributor

oritwas commented Sep 3, 2015

looks good

ghost pushed a commit that referenced this pull request Sep 3, 2015

Loic Dachary
Merge pull request #5716 from dachary/wip-12851-hammer
Ensure that swift keys don't include backslashes

Reviewed-by: Orit Wasserman <owasserm@redhat.com>

@ghost ghost merged commit 317770d into ceph:hammer Sep 3, 2015

@ghost ghost changed the title from Ensure that swift keys don't include backslashes to rgw: Ensure that swift keys don't include backslashes Sep 13, 2015

This issue was closed.

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