Skip to content

Fix marshall error in S3Path.exists()#6706

Merged
bentsherman merged 2 commits intomasterfrom
6704-marshall-error-s3-exists
Jan 8, 2026
Merged

Fix marshall error in S3Path.exists()#6706
bentsherman merged 2 commits intomasterfrom
6704-marshall-error-s3-exists

Conversation

@jorgee
Copy link
Contributor

@jorgee jorgee commented Jan 8, 2026

close #6704
This pull request fixes a bug introduced in the porting to AWS SDK v2. The old getObjectMetadata was including buckets and objects. In AWS the headObject returns a marshall error if key is empty. There is the headBucket method to get the bucket metadata.

  • Added a new method getBucketMetadata to S3Client.java to fetch S3 bucket metadata using the AWS SDK's headBucket operation.

  • Updated S3ObjectSummaryLookup.java to use getBucketMetadata instead of getObjectMetadata when the S3 key is empty, ensuring correct handling of bucket-level metadata. The resulting S3Object no longer includes object-specific fields like eTag, lastModified, or contentLength when representing a bucket.

Signed-off-by: jorgee <jorge.ejarque@seqera.io>
@netlify
Copy link

netlify bot commented Jan 8, 2026

Deploy Preview for nextflow-docs-staging canceled.

Name Link
🔨 Latest commit 44b33c1
🔍 Latest deploy log https://app.netlify.com/projects/nextflow-docs-staging/deploys/695fd6471db9230008e73647

@bentsherman bentsherman merged commit c1bd552 into master Jan 8, 2026
23 checks passed
@bentsherman bentsherman deleted the 6704-marshall-error-s3-exists branch January 8, 2026 16:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Marshall error when checking attributes of an S3 bucket path

3 participants