Skip to content

[PB-5969]: skip zero-byte files in folder uploads to prevent blocking#1932

Open
terrerox wants to merge 5 commits intomasterfrom
fix/empty-files-folder
Open

[PB-5969]: skip zero-byte files in folder uploads to prevent blocking#1932
terrerox wants to merge 5 commits intomasterfrom
fix/empty-files-folder

Conversation

@terrerox
Copy link
Copy Markdown
Contributor

@terrerox terrerox commented Apr 15, 2026

Description

Filter out empty files before dispatching the folder upload thunk, and log partial failures instead of aborting the whole folder upload. This prevents zero-byte files (e.g. init.py) from stalling or failing
the entire directory upload.

Related Issues

Related Pull Requests

Checklist

  • Changes have been tested locally.
  • Unit tests have been written or updated as necessary.
  • The code adheres to the repository's coding standards.
  • Relevant documentation has been added or updated.
  • No new warnings or errors have been introduced.
  • SonarCloud issues have been reviewed and addressed.
  • QA Passed

Testing Process

Additional Notes

@terrerox terrerox self-assigned this Apr 15, 2026
@terrerox terrerox requested a review from a team as a code owner April 15, 2026 02:58
@terrerox terrerox marked this pull request as draft April 15, 2026 02:58
@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented Apr 15, 2026

Deploying drive-web with  Cloudflare Pages  Cloudflare Pages

Latest commit: 23187a2
Status: ✅  Deploy successful!
Preview URL: https://3c35102a.drive-web.pages.dev
Branch Preview URL: https://fix-empty-files-folder.drive-web.pages.dev

View logs

@terrerox terrerox force-pushed the fix/empty-files-folder branch from 1606e95 to 103b0fe Compare April 15, 2026 03:37
@terrerox terrerox force-pushed the fix/empty-files-folder branch from 103b0fe to c8742c1 Compare April 15, 2026 04:14
@sonarqubecloud
Copy link
Copy Markdown

@terrerox terrerox marked this pull request as ready for review April 15, 2026 19:43
Copy link
Copy Markdown
Collaborator

@CandelR CandelR left a comment

Choose a reason for hiding this comment

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

Before reviewing the PR, this flow had already been removed. Has someone from Product requested that it be added back? I can see that this task is linked to a bug and there hasn’t been a request to reintroduce this behaviour, which is why I’m asking

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