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: fix the bug that the s3cmd abortmp operation may delete the bucket by mistake. #38731
Conversation
@mattbenjamin @cbodley Please take a review. Thanks |
@houfei001 Thank you for your contribution.
Maybe add @houfei001 what do you think? |
@ofriedma That's a good point. I'll try to deal with it. |
…et by mistake. Fixes: https://tracker.ceph.com/issues/48727 Signed-off-by: houfei <houfei@chinatelecom.cn>
@ofriedma Happy New Year. |
In other words, we only allow legal args to pass through. |
I've tested it, it's OK, and this modification has minimal impact on the existing code. Would you mind to take a review again. @ofriedma |
@houfei001 nice work when illegal args exist, maybe It's better to add this check for object delete handler as well. |
This pull request has been automatically marked as stale because it has not had any activity for 60 days. It will be closed if no further activity occurs for another 30 days. |
This pull request can no longer be automatically merged: a rebase is needed and changes have to be manually resolved |
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! |
Fixes: https://tracker.ceph.com/issues/48727
Signed-off-by: houfei houfei@chinatelecom.cn