fix test

Author: DmitriyMusatkin

source/s3_auto_ranged_get.c

@@ -266,7 +266,7 @@ static bool s_s3_auto_ranged_get_update(
                             AWS_S3_REQUEST_FLAG_RECORD_RESPONSE_HEADERS |
                                 AWS_S3_REQUEST_FLAG_ALLOCATE_BUFFER_FROM_POOL);
                         /* Note: our current default logic is to do part 1, discover size and then abort if payload its
-                         * too huge We optimistically reserve part size for it */
+                         * too huge We optimistically reserve part size for it */                
                         request->part_range_start = 0;
                         request->part_range_end = meta_request->part_size - 1;
                         ++auto_ranged_get->synced_data.num_parts_requested;

source/s3_auto_ranged_put.c

@@ -637,6 +637,9 @@ static bool s_s3_auto_ranged_put_update(
 
                 s_compute_request_body_size(meta_request, request);
 
+                AWS_LOGF_DEBUG(0, "foo %zu %zu %zu", request->content_length, 
+                    request->part_range_start, request->part_range_end);
+
                 /* If request was previously uploaded, we prepare it to ensure checksums still match,
                  * but ultimately it gets marked no-op and we don't send it */
                 request->was_previously_uploaded = request_previously_uploaded;

source/s3_default_buffer_pool.c

@@ -649,7 +649,6 @@ struct aws_s3_default_buffer_ticket *s_try_reserve_synced(
         overall_taken -= buffer_pool->forced_used - max_impact_of_forced_on_limit;
     }
 
-    AWS_LOGF_DEBUG(0, "foo");
     if ((meta.size + overall_taken) <= buffer_pool->mem_limit) {
         ticket = aws_mem_calloc(buffer_pool->base_allocator, 1, sizeof(struct aws_s3_default_buffer_ticket));
         ticket->size = meta.size;
@@ -663,7 +662,6 @@ struct aws_s3_default_buffer_ticket *s_try_reserve_synced(
         } else if (meta.size <= buffer_pool->primary_size_cutoff) {
             /* This needs to be smarter. Currently if primary req size is below limit, it will allocate full block,
                 which can be above limit. */
-            AWS_LOGF_DEBUG(0, "foo1");
             if (meta.size <= buffer_pool->primary_size_min_cutoff) {
                 ticket->reserved_from = AWS_S3_BUFFER_POOL_RESERVED_FROM_SECONDARY;
                 buffer_pool->secondary_reserved += meta.size;
@@ -687,7 +685,6 @@ struct aws_s3_default_buffer_ticket *s_try_reserve_synced(
                 }
             }
         } else {
-            AWS_LOGF_DEBUG(0, "foo2");
             ticket->reserved_from = AWS_S3_BUFFER_POOL_RESERVED_FROM_SECONDARY;
             buffer_pool->secondary_reserved += meta.size;
         }

source/s3_meta_request.c

@@ -1425,8 +1425,13 @@ static int s_s3_meta_request_incoming_headers(
                     }
                     if (request->part_range_end != object_range_end) {
                         /* In the case where the object size is less than the range requested. It must be return the
-                         * last part to the end of the object. */
-                        if (object_size != object_range_end + 1 || request->part_range_end < object_range_end) {
+                         * last part to the end of the object. 
+                         * Note: that when get with part 1 is used we dont know how big the buffer will be. We optimistically allocate
+                         * part sized buffer, and see if its enough. If its over, the req will get canceled. So in that case skip validation
+                         * on expected size.
+                         */
+                        if (request->request_tag != AWS_S3_AUTO_RANGE_GET_REQUEST_TYPE_GET_OBJECT_WITH_PART_NUMBER_1 && 
+                            (object_size != object_range_end + 1 || request->part_range_end < object_range_end)) {
                             /* Something went wrong if it's matching. Log the error. */
                             AWS_LOGF_ERROR(
                                 AWS_LS_S3_META_REQUEST,

tests/s3_data_plane_tests.c

@@ -4131,8 +4131,8 @@ static int s_test_s3_round_trip_default_get_fc_helper(
                     .object_path_override = object_path,
                 },
         };
-        if (algorithm == AWS_SCA_CRC32 || algorithm == AWS_SCA_CRC32C || algorithm == AWS_SCA_CRC64NVME) {
-            /* Full object checksums doesn't support SHA or XXHASH. */
+        if (algorithm != AWS_SCA_SHA1 && algorithm != AWS_SCA_SHA256) {
+            /* Full object checksums doesn't support SHA. */
             put_options.put_options.full_object_checksum = full_object_checksum;
         }