default request without body

Author: TingDaoK

source/s3_client.c

@@ -1236,7 +1236,7 @@ static struct aws_s3_meta_request *s_s3_client_meta_request_factory_default(
                                 client->allocator,
                                 client,
                                 AWS_S3_REQUEST_TYPE_GET_OBJECT,
-                                content_length,
+                                0, /* Content length for request */
                                 false /*should_compute_content_md5*/,
                                 options);
                         }

source/s3_default_meta_request.c

@@ -329,6 +329,7 @@ static void s_s3_default_prepare_request_finish(
 
     struct aws_s3_request *request = request_prep->request;
     struct aws_s3_meta_request *meta_request = request->meta_request;
+    struct aws_s3_meta_request_default *meta_request_default = meta_request->impl;
 
     if (error_code != AWS_ERROR_SUCCESS) {
         goto finish;
@@ -347,13 +348,17 @@ static void s_s3_default_prepare_request_finish(
         struct aws_http_headers *headers = aws_http_message_get_headers(message);
         aws_http_headers_set(headers, g_request_validation_mode, g_enabled);
     }
-    aws_s3_message_util_assign_body(
-        meta_request->allocator,
-        &request->request_body,
-        message,
-        &meta_request->checksum_config,
-        NULL /* out_checksum */);
-
+    if (meta_request_default->request_type == AWS_S3_REQUEST_TYPE_PUT_OBJECT ||
+        meta_request_default->request_type == AWS_S3_REQUEST_TYPE_UPLOAD_PART || request->request_body.len > 0) {
+        /* Only PUT Object and Upload part support trailing checksum, that needs the special encoding even if the body
+         * has 0 length. */
+        aws_s3_message_util_assign_body(
+            meta_request->allocator,
+            &request->request_body,
+            message,
+            &meta_request->checksum_config,
+            NULL /* out_checksum */);
+    }
     aws_s3_request_setup_send_data(request, message);
 
     aws_http_message_release(message);

tests/CMakeLists.txt

@@ -176,6 +176,7 @@ add_net_test_case(test_s3_round_trip)
 add_net_test_case(test_s3_round_trip_default_get)
 add_net_test_case(test_s3_round_trip_multipart_get_fc)
 add_net_test_case(test_s3_round_trip_default_get_fc)
+add_net_test_case(test_s3_round_trip_empty_fc)
 add_net_test_case(test_s3_round_trip_mpu_multipart_get_fc)
 add_net_test_case(test_s3_round_trip_mpu_multipart_get_with_list_algorithm_fc)
 add_net_test_case(test_s3_round_trip_mpu_default_get_fc)

tests/s3_data_plane_tests.c

@@ -3922,6 +3922,7 @@ static int s_test_s3_round_trip_default_get_fc_helper(
     struct aws_allocator *allocator,
     void *ctx,
     bool via_header,
+    uint32_t object_size_mb,
     enum aws_s3_tester_full_object_checksum full_object_checksum) {
     (void)ctx;
 
@@ -3943,8 +3944,9 @@ static int s_test_s3_round_trip_default_get_fc_helper(
         snprintf(
             object_path_sprintf_buffer,
             sizeof(object_path_sprintf_buffer),
-            "/prefix/round_trip/test_default_fc_%d.txt",
-            algorithm);
+            "/prefix/round_trip/test_default_fc_%d_%d.txt",
+            algorithm,
+            object_size_mb);
 
         ASSERT_SUCCESS(aws_s3_tester_upload_file_path_init(
             allocator, &path_buf, aws_byte_cursor_from_c_str(object_path_sprintf_buffer)));
@@ -3960,7 +3962,7 @@ static int s_test_s3_round_trip_default_get_fc_helper(
             .checksum_via_header = via_header,
             .put_options =
                 {
-                    .object_size_mb = 1,
+                    .object_size_mb = object_size_mb,
                     .object_path_override = object_path,
                 },
         };
@@ -4000,18 +4002,22 @@ static int s_test_s3_round_trip_default_get_fc_helper(
 
 AWS_TEST_CASE(test_s3_round_trip_default_get_fc, s_test_s3_round_trip_default_get_fc)
 static int s_test_s3_round_trip_default_get_fc(struct aws_allocator *allocator, void *ctx) {
-    return s_test_s3_round_trip_default_get_fc_helper(allocator, ctx, false, AWS_TEST_FOC_NONE);
+    return s_test_s3_round_trip_default_get_fc_helper(allocator, ctx, false, 1 /*object_size_mb*/, AWS_TEST_FOC_NONE);
+}
+AWS_TEST_CASE(test_s3_round_trip_empty_fc, s_test_s3_round_trip_empty_fc)
+static int s_test_s3_round_trip_empty_fc(struct aws_allocator *allocator, void *ctx) {
+    return s_test_s3_round_trip_default_get_fc_helper(allocator, ctx, false, 0, AWS_TEST_FOC_NONE);
 }
 
 AWS_TEST_CASE(test_s3_round_trip_default_get_fc_header, s_test_s3_round_trip_default_get_fc_header)
 static int s_test_s3_round_trip_default_get_fc_header(struct aws_allocator *allocator, void *ctx) {
-    return s_test_s3_round_trip_default_get_fc_helper(allocator, ctx, true, AWS_TEST_FOC_NONE);
+    return s_test_s3_round_trip_default_get_fc_helper(allocator, ctx, true, 1, AWS_TEST_FOC_NONE);
 }
 AWS_TEST_CASE(
     test_s3_round_trip_default_get_full_object_checksum_fc,
     s_test_s3_round_trip_default_get_full_object_checksum_fc)
 static int s_test_s3_round_trip_default_get_full_object_checksum_fc(struct aws_allocator *allocator, void *ctx) {
-    return s_test_s3_round_trip_default_get_fc_helper(allocator, ctx, false, AWS_TEST_FOC_HEADER);
+    return s_test_s3_round_trip_default_get_fc_helper(allocator, ctx, false, 1, AWS_TEST_FOC_HEADER);
 }
 
 static int s_test_s3_round_trip_multipart_get_fc_helper(struct aws_allocator *allocator, void *ctx, bool via_header) {