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: s3 copy object return 'x-amz-copy-source-version-id' and 'x-amz-version-id' #20348
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good; could you create a tracker issue which this fixes, put tracker link in commit msg, and -consider- writing an s3-test which checks for these?
src/rgw/rgw_op.cc
Outdated
@@ -4551,15 +4555,15 @@ void RGWCopyObj::execute() | |||
&mtime, | |||
mod_ptr, | |||
unmod_ptr, | |||
high_precision_time, | |||
high_precision_time, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
spacing change
src/rgw/rgw_op.cc
Outdated
if_match, | ||
if_nomatch, | ||
attrs_mod, | ||
copy_if_newer, | ||
copy_if_newer, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
spacing change
src/rgw/rgw_op.cc
Outdated
attrs, RGW_OBJ_CATEGORY_MAIN, | ||
olh_epoch, | ||
(delete_at ? *delete_at : real_time()), | ||
(version_id.empty() ? NULL : &version_id), | ||
&dst_version_id, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what happens to the former NULL case?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think version_id.empty()
is enough, and it might be more proper to use string &version
instead of string *version
for RGWRados::copy_obj
6c29797
to
9444b04
Compare
…-version-id' in response Fixes: http://tracker.ceph.com/issues/22946 Signed-off-by: fang yuxiang <fang.yuxiang@eisoo.com>
@mattbenjamin |
@mattbenjamin |
ping @mattbenjamin @cbodley Could we do something for this? |
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. |
unstale me my man |
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. |
This pull request has been automatically closed because there has been no activity for 90 days. Please feel free to reopen this pull request (or open a new one) if the proposed change is still appropriate. Thank you for your contribution! |
No description provided.