Skip to content

chore(deps): bump opendal to 0.56, migrate to reqsign-core 3.0#2401

Open
kszucs wants to merge 2 commits intoapache:mainfrom
kszucs:bump/opendal-0.56
Open

chore(deps): bump opendal to 0.56, migrate to reqsign-core 3.0#2401
kszucs wants to merge 2 commits intoapache:mainfrom
kszucs:bump/opendal-0.56

Conversation

@kszucs
Copy link
Copy Markdown
Member

@kszucs kszucs commented May 4, 2026

Summary

Bumps opendal from 0.55 to 0.56, which is now published on crates.io.

The 0.56 release made two breaking API changes that required code updates:

  • opendal::Scheme was removed. OpenDalResolvingStorage used it as the HashMap key for caching per-scheme storage instances. Replaced with &'static str (e.g. "s3", "gcs"), with S3-family aliases (s3a, s3n) and GCS aliases (gs) still collapsing to a single entry.

  • reqsign was split into focused crates. The old reqsign 0.16 monolith (with AwsCredentialLoad / reqwest::Client) was replaced by reqsign-aws-v4 + reqsign-core 3.0. CustomAwsCredentialLoader now wraps any ProvideCredential<Credential = AwsCredential> implementation instead of Arc<dyn AwsCredentialLoad>, which removes the Arc wrapping at call sites.

@kszucs kszucs force-pushed the bump/opendal-0.56 branch 2 times, most recently from 23f5d13 to 4659c06 Compare May 4, 2026 07:07
@kszucs kszucs force-pushed the bump/opendal-0.56 branch from 4659c06 to 91a9883 Compare May 4, 2026 07:35
format!("{relative_path}/")
};
Ok(op.remove_all(&path).await.map_err(from_opendal_error)?)
Ok(op
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove_all was deprecated

Copy link
Copy Markdown
Collaborator

@CTTY CTTY left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the contribution! left some minor feedbacks

Also please address the conflict

Comment thread crates/storage/opendal/tests/file_io_s3_test.rs Outdated
Comment thread crates/storage/opendal/Cargo.toml Outdated
@kszucs kszucs force-pushed the bump/opendal-0.56 branch from 91a9883 to 0b76c12 Compare May 5, 2026 13:06
@kszucs
Copy link
Copy Markdown
Member Author

kszucs commented May 5, 2026

@CTTY ready for another review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants