Skip to content

Comments

Normalize object store paths for writeStream#58062

Open
Yaroslav98214 wants to merge 1 commit intonextcloud:masterfrom
Yaroslav98214:fix-objectstore-path-normalization
Open

Normalize object store paths for writeStream#58062
Yaroslav98214 wants to merge 1 commit intonextcloud:masterfrom
Yaroslav98214:fix-objectstore-path-normalization

Conversation

@Yaroslav98214
Copy link

Summary

  • Normalize paths in ObjectStoreStorage::writeStream to prevent double-slash cache keys
  • Add regression test for path normalization

Impact

Prevents object-store cache mismatches and missing file lookups when paths contain double slashes.

@Yaroslav98214 Yaroslav98214 requested a review from a team as a code owner February 4, 2026 21:03
@Yaroslav98214 Yaroslav98214 requested review from CarlSchwan, icewind1991, leftybournes and salmart-dev and removed request for a team February 4, 2026 21:03
@Yaroslav98214 Yaroslav98214 force-pushed the fix-objectstore-path-normalization branch from dfa0d92 to 4c9fcfe Compare February 4, 2026 21:05
Signed-off-by: Yaroslav98214 <diakovichyaroslav30@gmail.com>
@Yaroslav98214 Yaroslav98214 force-pushed the fix-objectstore-path-normalization branch from 4c9fcfe to 255cdb2 Compare February 5, 2026 01:28
@github-actions
Copy link
Contributor

Hello there,
Thank you so much for taking the time and effort to create a pull request to our Nextcloud project.

We hope that the review process is going smooth and is helpful for you. We want to ensure your pull request is reviewed to your satisfaction. If you have a moment, our community management team would very much appreciate your feedback on your experience with this PR review process.

Your feedback is valuable to us as we continuously strive to improve our community developer experience. Please take a moment to complete our short survey by clicking on the following link: https://cloud.nextcloud.com/apps/forms/s/i9Ago4EQRZ7TWxjfmeEpPkf6

Thank you for contributing to Nextcloud and we hope to hear from you soon!

(If you believe you should not receive this message, you can add yourself to the blocklist.)

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.

2 participants