Luka-J9 opened a new issue, #1245:
URL: https://github.com/apache/pekko-connectors/issues/1245

   An extension of: 
   https://github.com/apache/pekko-connectors/pull/81
   
   When `x-amz-storage-class` is manually added to the headers of a 
`multipartCopy` request the resulting call fails with a 
   
   ```
   S3Exception: The specified header is not valid in this context
   ```
   
   I believe the fix is the same as what was in the original PR, however it was 
not done for the CopyPart
   
   Changing the code to follow the pattern of the linked PR resolves the issue 
- however it's worth noting that this does come at the expense of other 
legitimate fields like `x-amz-request-payer`
   
   [See AWS docs for valid 
headers](https://docs.aws.amazon.com/AmazonS3/latest/API/API_UploadPartCopy.html#API_UploadPartCopy_RequestSyntax)
   
   It seems that we would need to filter out the headers that are not in this 
list, rather than only include the SSE headers.
   
   Happy to contribute - although I'm not exactly sure what would be the 
pattern to employ to only allow for a subset of headers


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to