s3s/crates/s3s/src/ops/mod.rs

Lines 378 to 402 in cb67432

    
           let (op, needs_full_body) = 'resolve: { 
        
               if let Some(multipart) = &mut req.s3ext.multipart { 
        
                   if req.method == Method::POST { 
        
                       match s3_path { 
        
                           S3Path::Root => return Err(unknown_operation()), 
        
                           S3Path::Bucket { .. } => { 
        
                               // POST object 
        
                               debug!(?multipart); 
        
                               let file_stream = multipart.take_file_stream().expect("missing file stream"); 
        
                               // Aggregate file stream with size limit to get known length 
        
                               // This is required because downstream handlers (like s3s-proxy) need content-length 
        
                               let vec_bytes = http::aggregate_file_stream_limited(file_stream, http::MAX_POST_OBJECT_FILE_SIZE) 
        
                                   .await 
        
                                   .map_err(|e| invalid_request!(e, "failed to read file stream"))?; 
        
                               let vec_stream = crate::stream::VecByteStream::new(vec_bytes); 
        
                               req.s3ext.vec_stream = Some(vec_stream); 
        
                               break 'resolve (&PutObject as &'static dyn Operation, false); 
        
                           } 
        
                           // FIXME: POST /bucket/key hits this branch 
        
                           S3Path::Object { .. } => return Err(s3_error!(MethodNotAllowed)), 
        
                       } 
        
                   } 
        
               } 
        
               resolve_route(req, s3_path, req.s3ext.qs.as_ref())? 
        
           };

	let (op, needs_full_body) = 'resolve: {
	if let Some(multipart) = &mut req.s3ext.multipart {
	if req.method == Method::POST {
	match s3_path {
	S3Path::Root => return Err(unknown_operation()),
	S3Path::Bucket { .. } => {
	// POST object
	debug!(?multipart);
	let file_stream = multipart.take_file_stream().expect("missing file stream");
	// Aggregate file stream with size limit to get known length
	// This is required because downstream handlers (like s3s-proxy) need content-length
	let vec_bytes = http::aggregate_file_stream_limited(file_stream, http::MAX_POST_OBJECT_FILE_SIZE)
	.await
	.map_err(\|e\| invalid_request!(e, "failed to read file stream"))?;
	let vec_stream = crate::stream::VecByteStream::new(vec_bytes);
	req.s3ext.vec_stream = Some(vec_stream);
	break 'resolve (&PutObject as &'static dyn Operation, false);
	}
	// FIXME: POST /bucket/key hits this branch
	S3Path::Object { .. } => return Err(s3_error!(MethodNotAllowed)),
	}
	}
	}
	resolve_route(req, s3_path, req.s3ext.qs.as_ref())?
	};

Missing support for success_action_status and success_action_redirect in S3 POST uploads #425

Description

Metadata

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Issue actions

Missing support for `success_action_status` and `success_action_redirect` in S3 POST uploads #425