apache / apache/opendal

new feature: allow inject a customized signer for aws s3

Open
#4,960 8 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
Rust
Stars
5.4k
Forks
825
Avg merge
1d 14m
Merged PRs (30d)
127

Description

### Feature Description

background: https://github.com/apache/iceberg-rust/issues/506

> It seems we need to implement this signer in iceberg-rust and pass it as a customized signer to opendal.

### Problem and Solution

but currently the signer is a struct `AwsV4Signer`.

we need make the signer in `S3Core` a trait before allowing it customizable.

maybe we can defined a trait called `S3Signer` in opendal, and make `AwsV4Signer` as it's implementation by default.

### Additional Context

_No response_

### Are you willing to contribute to the development of this feature?

- [X] Yes, I am willing to contribute to the development of this feature.

Contributor guide

Open the contributing guide

Research direction

Start by reading the S3Core and AwsV4Signer implementations in OpenDAL, then review the linked iceberg-rust issue for how a custom signer needs to be passed through. Done means S3Core accepts an injectable signer abstraction while AwsV4Signer remains the default implementation.

Written by the indexing model from the issue text.

Assessment

Tech stack
aws, rust
Domain
cloud
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.