Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

update_lookup_table ignores files that have been split. #10

Closed
BURG3R5 opened this issue Aug 14, 2022 · 0 comments · Fixed by #13
Closed

update_lookup_table ignores files that have been split. #10

BURG3R5 opened this issue Aug 14, 2022 · 0 comments · Fixed by #13
Assignees
Labels
bug Something isn't working storage

Comments

@BURG3R5
Copy link
Owner

BURG3R5 commented Aug 14, 2022

  • Suppose a bucket has a file size of 150 bytes and the homeserver's file size limit is set to 100 bytes.
  • The IndexStorage class' split_large_buckets will correctly divide this bucket into two "fraction-of-bucket"s of 100 and 50 bytes.
  • The user will upload these files, the callback will store them in the mxc_uris_map.

The problem is that update_lookup_table converts each location into a single other location. So the bucket's location will be converted to just the first file.

The solution is for convert_location to find and return all the remote files matching the location.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working storage
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant