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.

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

Fix: precondition parsing issue #516

Merged
merged 1 commit into from
Apr 22, 2020

Conversation

mervinkid
Copy link
Member

What this PR does / why we need it:
Fix the parsing issue of the two preconditions of If-Match and
If-None-Match.

We found that the request sent through the AWS S3 SDK included
double quotes for the values of these two preconditions.

This part is not specified in the AWS S3 API Reference.

Affected APIs:

  • HeadObject
  • GetObject

Which issue this PR fixes:
NONE

Special notes for your reviewer:
NONE.

Release note:
NONE.

@mervinkid mervinkid added interface/s3 Topic about object storage interface component/objectnode Topic about ObjectNode priority/high labels Apr 22, 2020
@mervinkid mervinkid force-pushed the fix_precondition_parsing branch 2 times, most recently from 1a952c8 to fdf8f28 Compare April 22, 2020 08:34
Fix the parsing issue of the two preconditions of If-Match and
If-None-Match.

We found that the request sent through the AWS S3 SDK included
double quotes for the values of these two preconditions.

This part is not specified in the AWS S3 API Reference.

Affected APIs: HeadObject, GetObject

Signed-off-by: Mofei Zhang <mofei2816@gmail.com>
@mervinkid mervinkid force-pushed the fix_precondition_parsing branch from fdf8f28 to 5c7c998 Compare April 22, 2020 09:25
Copy link
Member

@awzhgw awzhgw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

review+1

@awzhgw awzhgw merged commit 9b60835 into cubefs:master Apr 22, 2020
awzhgw pushed a commit that referenced this pull request Mar 30, 2023
enhance: ec create partition choose ecnode of partitionSize < maxDiskAvailSize
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/objectnode Topic about ObjectNode interface/s3 Topic about object storage interface priority/high
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants