Skip to content

fix(core): preserve whitespace in paths#7640

Draft
georgeorange-crypto wants to merge 1 commit into
apache:mainfrom
georgeorange-crypto:codex/preserve-path-whitespace
Draft

fix(core): preserve whitespace in paths#7640
georgeorange-crypto wants to merge 1 commit into
apache:mainfrom
georgeorange-crypto:codex/preserve-path-whitespace

Conversation

@georgeorange-crypto
Copy link
Copy Markdown

Summary

  • stop trimming leading and trailing whitespace in operation paths
  • keep existing normalization for leading slashes and repeated slashes
  • add a behavior test that writes and reads a file whose name ends with a space

Closes #6577.

Validation

  • cargo fmt --check
  • cargo test -q -p opendal-core raw::path::tests::test_normalize_path
  • OPENDAL_TEST=fs cargo test -q -p opendal --features services-fs,tests --test behavior -- test_read_with_trailing_space
  • cargo check -q

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.

bug: fail to read files with trailing whitespaces

1 participant