Skip to content

Add file size validation.#983

Open
balagan73 wants to merge 5 commits intomainfrom
957-upload-big-docs
Open

Add file size validation.#983
balagan73 wants to merge 5 commits intomainfrom
957-upload-big-docs

Conversation

@balagan73
Copy link
Copy Markdown
Contributor

Add file size validation.

@balagan73 balagan73 linked an issue Jan 30, 2026 that may be closed by this pull request
@balagan73
Copy link
Copy Markdown
Contributor Author

image

The ajax error still occurs, but we have an informative error message on the UI.

@balagan73 balagan73 requested a review from amitaibu January 30, 2026 16:09
@amitaibu
Copy link
Copy Markdown
Member

How come Drupal's file size validation doesn't kick in?

@balagan73
Copy link
Copy Markdown
Contributor Author

@amitaibu I tried with pure drupal core, and it did not have a size validation. It only threw the ajax error, with detailed information only in the console. It seems Drupal never had a client side file size validation.
https://www.drupal.org/project/drupal/issues/2533896
Drupal has a file exension validation though working on the client side, that's what this PR is based on.
There is a contrib module, but currently it is not working properly. Maybe the right way is to move this to this logic to this contrib.

@amitaibu
Copy link
Copy Markdown
Member

On a clean Drupal installation, there's no file validation? Doesn't sound right

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.

Uploading 100MB+ document throws ajax error

2 participants