Skip to content

fix(metadata): Correct filesize limit calculation for metadata genera…#54100

Closed
enjeck wants to merge 1 commit into
masterfrom
update-size-byte
Closed

fix(metadata): Correct filesize limit calculation for metadata genera…#54100
enjeck wants to merge 1 commit into
masterfrom
update-size-byte

Conversation

@enjeck

@enjeck enjeck commented Jul 28, 2025

Copy link
Copy Markdown
Contributor

…tion

Summary

The value of the metadata_max_filesize parameter (or its default value), which is expressed in megabytes, is compared to the file size in bytes for which metadata needs to be generated. However, the conversion from megabytes to bytes is performed by multiplying by 1000000 (1000 * 1000) instead of 1048576 (1024 * 1024).

This means that we believe, by default, the system generates metadata for files smaller than 268435456 bytes (256 MB), when in fact, it's for files up to 256000000 bytes (244 MB).

Checklist

…tion

Signed-off-by: Enjeck C <patrathewhiz@gmail.com>
@enjeck enjeck self-assigned this Jul 28, 2025
@enjeck enjeck requested a review from a team as a code owner July 28, 2025 04:44
@enjeck enjeck requested review from ArtificialOwl, sorbaugh and yemkareems and removed request for a team July 28, 2025 04:44
@skjnldsv skjnldsv added bug 2. developing Work in progress labels Aug 19, 2025
@joshtrichards

Copy link
Copy Markdown
Member

Superseded by #54951; closing as no longer necessary.

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

Labels

2. developing Work in progress bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants