Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: zip-rs/zip2
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: v4.6.0
Choose a base ref
...
head repository: zip-rs/zip2
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: v5.1.1
Choose a head ref
  • 17 commits
  • 34 files changed
  • 12 contributors

Commits on Sep 2, 2025

  1. fix: Swap to liblzma from lzma-rs introduced issue (#407)

    * AI Generated
    
    * Delete LZMA_FIX_SUMMARY.md
    
    Signed-off-by: Chris Hennick <[email protected]>
    
    * Move new test into tests directory
    
    * style: cargo fmt --all
    
    ---------
    
    Signed-off-by: Chris Hennick <[email protected]>
    Co-authored-by: amazon-q-developer[bot] <208079219+amazon-q-developer[bot]@users.noreply.github.com>
    Co-authored-by: Chris Hennick <[email protected]>
    Co-authored-by: hennickc <[email protected]>
    3 people authored Sep 2, 2025
    Configuration menu
    Copy the full SHA
    8b7eb9b View commit details
    Browse the repository at this point in the history

Commits on Sep 3, 2025

  1. chore: release v4.6.1 (#409)

    * chore: release v4.6.1
    
    * Update CHANGELOG.md
    
    Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
    Signed-off-by: Chris Hennick <[email protected]>
    
    ---------
    
    Signed-off-by: Chris Hennick <[email protected]>
    Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
    Pr0methean and gemini-code-assist[bot] authored Sep 3, 2025
    Configuration menu
    Copy the full SHA
    56fa5f6 View commit details
    Browse the repository at this point in the history
  2. Use native implementation of LZMA and XZ (#405)

    * deps: Use native lzma-rust2 instead of liblzma
    
    This is a native port of the XZ's liblzma. The main advantage is, that the crate is native Rust and is also used by the 7z crate.
    
    Performance should be equal to the liblzma crate, since I spend quite a bit of time to improve the performance.
    
    I only used the single threaded version of the reader/writer, since parallelization was not enabled for liblzma. lzma-rust2 has multithreaded reader/writer though.
    
    I had to remove the old bug report fix, since lzma_rust2 doesn't have the reported behavior. The test case also was obviously AI generated and useless.
    
    * fix: Properly implement LZMA decoding
    
    The old implementation actually didn't worked. The test file uses STORE as compression. This makes sure that LZMA is properly implemented.
    
    I remembered that I had to defer the initialization of reading the properties when implementing PPMd, so I did this here too.
    
    * fix: Fix linter issue
    
    * fix: Formatting of README.md
    
    * fix: Reduce IO reads
    
    * fix: Move XZ decoder and encoder to the heap
    
    * fix: Fix code review issues
    
    * fix: Fix UnwindSafe issue by using latest lzma-rust2 version
    
    * Rephrase a comment in src/compression.rs
    
    Signed-off-by: Chris Hennick <[email protected]>
    
    ---------
    
    Signed-off-by: Chris Hennick <[email protected]>
    Co-authored-by: Chris Hennick <[email protected]>
    hasenbanck and Pr0methean authored Sep 3, 2025
    Configuration menu
    Copy the full SHA
    aaaa26f View commit details
    Browse the repository at this point in the history

Commits on Sep 4, 2025

  1. Potential fix for code scanning alert no. 205: Workflow does not cont…

    …ain permissions (#411)
    
    Signed-off-by: Chris Hennick <[email protected]>
    Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
    Configuration menu
    Copy the full SHA
    2f202c6 View commit details
    Browse the repository at this point in the history
  2. test(ci): Create rust-clippy.yml (#413)

    * Create rust-clippy.yml
    
    Signed-off-by: Chris Hennick <[email protected]>
    
    * Update .github/workflows/rust-clippy.yml
    
    Co-authored-by: amazon-q-developer[bot] <208079219+amazon-q-developer[bot]@users.noreply.github.com>
    Signed-off-by: Chris Hennick <[email protected]>
    
    * Update .github/workflows/rust-clippy.yml
    
    Co-authored-by: amazon-q-developer[bot] <208079219+amazon-q-developer[bot]@users.noreply.github.com>
    Signed-off-by: Chris Hennick <[email protected]>
    
    ---------
    
    Signed-off-by: Chris Hennick <[email protected]>
    Co-authored-by: amazon-q-developer[bot] <208079219+amazon-q-developer[bot]@users.noreply.github.com>
    Pr0methean and amazon-q-developer[bot] authored Sep 4, 2025
    Configuration menu
    Copy the full SHA
    82178ed View commit details
    Browse the repository at this point in the history
  3. chore(deps): update nt-time requirement from 0.10.6 to 0.12.1 (#414)

    Updates the requirements on [nt-time](https://github.com/sorairolake/nt-time) to permit the latest version.
    - [Changelog](https://github.com/sorairolake/nt-time/blob/develop/CHANGELOG.adoc)
    - [Commits](sorairolake/nt-time@v0.10.6...v0.12.1)
    
    ---
    updated-dependencies:
    - dependency-name: nt-time
      dependency-version: 0.12.1
      dependency-type: direct:production
    ...
    
    Signed-off-by: dependabot[bot] <[email protected]>
    Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
    dependabot[bot] authored Sep 4, 2025
    Configuration menu
    Copy the full SHA
    8a6e832 View commit details
    Browse the repository at this point in the history

Commits on Sep 5, 2025

  1. feat: Implement by_path*() methods on ZipArchive (#382)

    * Implement by_path*() methods.
    
    * Refactor by_path tests.
    
    * fix: test requires aes-crypto
    
    * fix: Move `use zip::AesMode` and declaration of `PASSWORD` into the test that uses them, since they're not used when the `aes-crypto` feature isn't enabled
    
    ---------
    
    Co-authored-by: Chris Hennick <[email protected]>
    Co-authored-by: hennickc <[email protected]>
    3 people authored Sep 5, 2025
    Configuration menu
    Copy the full SHA
    68882b2 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    e1d74a0 View commit details
    Browse the repository at this point in the history

Commits on Sep 9, 2025

  1. fix: aes was not copied correctly in raw copy methods (#417)

    * fix: aes encrypted files raw copy
    
    * fix: preserve aes header data for raw copy
    im7mortal authored Sep 9, 2025
    Configuration menu
    Copy the full SHA
    b611a59 View commit details
    Browse the repository at this point in the history
  2. chore: release v5.0.1 (#418)

    * chore: release v5.0.1
    
    * Update CHANGELOG.md
    
    Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
    Signed-off-by: Chris Hennick <[email protected]>
    
    ---------
    
    Signed-off-by: Chris Hennick <[email protected]>
    Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
    Pr0methean and gemini-code-assist[bot] authored Sep 9, 2025
    Configuration menu
    Copy the full SHA
    12c87d1 View commit details
    Browse the repository at this point in the history

Commits on Sep 10, 2025

  1. feat: Add legacy shrink/reduce/implode compression (#303)

    * Added legacy shrink/reduce/implode compression.
    
    * Remove the unused 0th len_count entry
    
    Signed-off-by: Chris Hennick <[email protected]>
    
    * Make legacy modules `pub(crate)` rather than `pub`
    
    Signed-off-by: Chris Hennick <[email protected]>
    
    * chore(deps): Bump MSRV to 1.83.0 for bistream-io
    
    * style: cargo fmt --all
    
    * style: cargo clippy --fix
    
    * chore: fix remaining Clippy warnings manually
    
    * style: cargo fmt --all
    
    * fix: errors introduced by fixing clippy warnings
    
    * fix: test failures on Windows due to newline style
    
    * fix: must define legacy compression methods unconditionally, for backward compatibility
    
    * style: Add missing doc comments
    
    * fix: doc comments apparently can't be before `cfg`
    
    * fix: doc comments apparently can't be before `cfg`
    
    * fix: Windows newline issues persisted (files not recognized as text?)
    
    * fix: Windows newline issues persisted (need to mark some files text and others binary)
    
    * fix?! Test a peer's suggested fix
    
    * fix use lf instead of crlf to fix tests on windows
    
    * Fix? make tests/data/legacy/*.out explicitly CRLF
    
    ---------
    
    Signed-off-by: Chris Hennick <[email protected]>
    Co-authored-by: Chris Hennick <[email protected]>
    Co-authored-by: hennickc <[email protected]>
    Co-authored-by: Speedy_Lex <[email protected]>
    4 people authored Sep 10, 2025
    Configuration menu
    Copy the full SHA
    0abee77 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    e341b3d View commit details
    Browse the repository at this point in the history

Commits on Sep 11, 2025

  1. chore(deps): update nt-time requirement from 0.10.6 to 0.12.1 (#416)

    Updates the requirements on [nt-time](https://github.com/sorairolake/nt-time) to permit the latest version.
    - [Changelog](https://github.com/sorairolake/nt-time/blob/develop/CHANGELOG.adoc)
    - [Commits](sorairolake/nt-time@v0.10.6...v0.12.1)
    
    ---
    updated-dependencies:
    - dependency-name: nt-time
      dependency-version: 0.12.1
      dependency-type: direct:production
    ...
    
    Signed-off-by: dependabot[bot] <[email protected]>
    Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
    Co-authored-by: Chris Hennick <[email protected]>
    dependabot[bot] and Pr0methean authored Sep 11, 2025
    Configuration menu
    Copy the full SHA
    4802b87 View commit details
    Browse the repository at this point in the history
  2. fix: Restore original file timestamp when unzipping with chrono (#46)

    * Set filetime
    
    * Add chrono
    
    * fix: Duplicate "chrono" entry in Cargo.toml
    
    Signed-off-by: Chris Hennick <[email protected]>
    
    * fix: typo in read.rs
    
    Signed-off-by: Chris Hennick <[email protected]>
    
    * fix: missing feature gate
    
    Signed-off-by: Chris Hennick <[email protected]>
    
    * Set filetime
    
    * Add chrono
    
    * fix: Duplicate "chrono" entry in Cargo.toml
    
    Signed-off-by: Chris Hennick <[email protected]>
    
    * fix: typo in read.rs
    
    Signed-off-by: Chris Hennick <[email protected]>
    
    * fix: Errors due to missing chrono feature gate
    
    * fix: bad merge
    
    * style: cargo fmt --all
    
    * fix: chrono has renamed method to from_utc
    
    * Revert "fix: chrono has renamed method to from_utc"
    
    This reverts commit 38cc3ca.
    
    * fix: from_utc is deprecated, but from_naive_utc_and_offset doesn't exist before 0.4.27
    
    ---------
    
    Signed-off-by: Chris Hennick <[email protected]>
    Co-authored-by: mass10 <[email protected]>
    Pr0methean and mass10 authored Sep 11, 2025
    Configuration menu
    Copy the full SHA
    5362be0 View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    8d094b8 View commit details
    Browse the repository at this point in the history
  4. fix: panic when reading empty extended-timestamp field (#404) (#422)

    * fix: panic when reading empty extended-timestamp field (#404)
    
    * Add test assertion message
    
    Co-authored-by: amazon-q-developer[bot] <208079219+amazon-q-developer[bot]@users.noreply.github.com>
    Signed-off-by: Chris Hennick <[email protected]>
    
    * Update src/extra_fields/extended_timestamp.rs
    
    Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
    Signed-off-by: Chris Hennick <[email protected]>
    
    * Update src/extra_fields/extended_timestamp.rs
    
    Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
    Signed-off-by: Chris Hennick <[email protected]>
    
    * Simplify test per Gemini suggestion
    
    ---------
    
    Signed-off-by: Chris Hennick <[email protected]>
    Co-authored-by: amazon-q-developer[bot] <208079219+amazon-q-developer[bot]@users.noreply.github.com>
    Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
    3 people authored Sep 11, 2025
    Configuration menu
    Copy the full SHA
    97c33a6 View commit details
    Browse the repository at this point in the history
  5. Configuration menu
    Copy the full SHA
    6423fee View commit details
    Browse the repository at this point in the history
Loading