Skip to content

feat: rewrite the archiver and added support for zstd and brotli #5283

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

Merged

Conversation

jagadam97
Copy link
Contributor

@jagadam97 jagadam97 commented Jul 12, 2025

Description

as @jayadattasai raised in #5279

http://github.com/mholt/archiver/v3 is deprecated and

and same developer(mholt) has created https://github.com/mholt/archives

this PR introcudes the replacement of above mentioned packages/Libraries.

along with that

  1. I have re-wrote few logic which will not have any side effect on functionality of the compressor while make the code cleaner, file and error handling appropriate.
  2. along with removing few comments.
  3. added support for brotli and zstd as these are also widely used compression standards.
Screenshot from 2025-07-12 15-10-39 Screenshot from 2025-07-12 15-11-19

Additional Information

Closes #5279

Checklist

Before submitting your PR, please indicate which issues are either fixed or closed by this PR. See GitHub Help: Closing issues using keywords.

  • I am aware the project is currently in maintenance-only mode. See README
  • I am aware that translations MUST be made through Transifex and that this PR is NOT a translation update
  • I am making a PR against the master branch.
  • I am sure File Browser can be successfully built. See builds and development.

@jagadam97 jagadam97 requested review from o1egl and hacdias as code owners July 12, 2025 09:39
@jagadam97 jagadam97 force-pushed the move-mholt-archiver-to-archives branch from bea4b6e to 26f26ca Compare July 12, 2025 09:51
@hacdias hacdias merged commit 7c71686 into filebrowser:master Jul 12, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants