From 96164c5936032b4edb973828178cc55793dd57cc Mon Sep 17 00:00:00 2001
From: Josh Triplett <josh@joshtriplett.org>
Date: Mon, 21 Apr 2025 02:53:12 +0100
Subject: [PATCH 1/2] Switch to zlib-rs by default and drop other zlib backends

As of zlib-rs 0.5.0 (depended on by flate2 1.1.1), zlib-rs no longer
exports C symbols by default, so it doesn't conflict with any other zlib
that might be loaded into the address space.

This removed the primary issue that made zlib selection a challenge
that needed to be exposed to users: users may already have some other
particular preference on zlib implementations, or want to use a system
library, or want to ensure C dependencies use a particular zlib. Since
zlib-rs 0.5.0 no longer conflicts with other zlib implementations, gix
can make this choice independently and not create issues for the user.

Given that, use zlib-rs by default, for performance out of the box with
no C compiler requirement, and deprecate all the feature flags for other
variations. A future major version bump can drop all of these features.

This also means that (for instance) max-performance becomes the same as
max-performance-safe, so such features are deprecated as well.

Depend on flate2 with `default-features = false`, which brings us closer
to eliminating the `miniz_oxide` dependency, which will improve build
time for everyone. Currently, the `gix-archive` dependency on `zip`
still activates the `zip/deflate` feature which enables
`flate2/rust_backend`; this can be fixed to use `deflate-flate2` as soon
as https://github.com/zip-rs/zip2/pull/340 is fixed upstream.

Fixes: https://github.com/GitoxideLabs/gitoxide/issues/1961
---
 .github/workflows/ci.yml      |  4 ++--
 .github/workflows/release.yml |  3 ---
 Cargo.lock                    | 25 ++------------------
 Cargo.toml                    | 40 ++++++++++----------------------
 README.md                     |  4 ++--
 etc/docker/Dockerfile.alpine  |  6 ++---
 gix-archive/Cargo.toml        |  9 ++++----
 gix-features/Cargo.toml       | 43 +++++++++++++++--------------------
 gix/Cargo.toml                | 32 ++++++++++----------------
 justfile                      |  4 ----
 tests/journey/gix.sh          |  9 +-------
 11 files changed, 55 insertions(+), 124 deletions(-)

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index 73d7a4b0d16..38daebcd927 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -67,7 +67,7 @@ jobs:
         continue-on-error: true
       - name: Check for unrecognized *-sys dependencies
         run: |
-          ! grep -qP '(?<!\blinux-raw)-sys\b' tree.txt
+          ! grep -qP '(?<!\b(linux-raw|libz-rs))-sys\b' tree.txt
       - name: Wrap cc1 (and cc1plus if present) to record calls
         run: |
           set -o noclobber  # Catch any collisions with existing entries in /usr/local.
@@ -402,7 +402,7 @@ jobs:
       - name: features of gix-features
         run: |
           set +x
-          for feature in progress parallel io-pipe crc32 zlib zlib-rust-backend cache-efficiency-debug; do
+          for feature in progress parallel io-pipe crc32 zlib cache-efficiency-debug; do
             (cd gix-features && cargo build --features "$feature" --target "$TARGET")
           done
       - name: crates with 'wasm' feature
diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml
index 0a2801a6e78..c47e9148ee9 100644
--- a/.github/workflows/release.yml
+++ b/.github/workflows/release.yml
@@ -231,7 +231,6 @@ jobs:
       - name: Install packages (Ubuntu)
         # Because openssl doesn't work on musl by default, we resort to max-pure.
         # And that won't need any dependency, so we can skip this or use `continue-on-error`.
-        # Once we want to support better zlib performance, we might have to re-add it.
         if: matrix.os == 'ubuntu-latest-disabled'
         run: |
           sudo apt-get update
@@ -541,8 +540,6 @@ jobs:
           msystem: MINGW${{ startsWith(matrix.target, 'i686-') && '32' || '64' }}
           pacboy: cc:p
           path-type: inherit
-      - name: Install prerequisites
-        run: vcpkg install zlib:x64-windows-static-md
       - name: 'Installation from crates.io: gitoxide'
         run: cargo +${{ matrix.rust }} install --target ${{ matrix.target }} --no-default-features --features max-pure --target-dir install-artifacts --debug --force gitoxide
         shell: msys2 {0}
diff --git a/Cargo.lock b/Cargo.lock
index 74337864f53..88ec78f0d2d 100644
--- a/Cargo.lock
+++ b/Cargo.lock
@@ -1131,9 +1131,7 @@ source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "7ced92e76e966ca2fd84c8f7aa01a4aea65b0eb6648d72f7c8f3e2764a67fece"
 dependencies = [
  "crc32fast",
- "libz-ng-sys",
  "libz-rs-sys",
- "libz-sys",
  "miniz_oxide",
 ]
 
@@ -3715,16 +3713,6 @@ dependencies = [
  "vcpkg",
 ]
 
-[[package]]
-name = "libz-ng-sys"
-version = "1.1.22"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "a7118c2c2a3c7b6edc279a8b19507672b9c4d716f95e671172dfa4e23f9fd824"
-dependencies = [
- "cmake",
- "libc",
-]
-
 [[package]]
 name = "libz-rs-sys"
 version = "0.5.0"
@@ -3741,7 +3729,6 @@ source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "8b70e7a7df205e92a1a4cd9aaae7898dac0aa555503cc0a649494d0d60e7651d"
 dependencies = [
  "cc",
- "cmake",
  "libc",
  "pkg-config",
  "vcpkg",
@@ -3787,12 +3774,6 @@ dependencies = [
  "scopeguard",
 ]
 
-[[package]]
-name = "lockfree-object-pool"
-version = "0.1.6"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "9374ef4228402d4b7e403e5838cb880d9ee663314b0a900d5a6aabf0c213552e"
-
 [[package]]
 name = "log"
 version = "0.4.26"
@@ -6504,14 +6485,12 @@ checksum = "868b928d7949e09af2f6086dfc1e01936064cc7a819253bce650d4e2a2d63ba8"
 
 [[package]]
 name = "zopfli"
-version = "0.8.1"
+version = "0.8.2"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "e5019f391bac5cf252e93bbcc53d039ffd62c7bfb7c150414d61369afe57e946"
+checksum = "edfc5ee405f504cd4984ecc6f14d02d55cfda60fa4b689434ef4102aae150cd7"
 dependencies = [
  "bumpalo",
  "crc32fast",
- "lockfree-object-pool",
  "log",
- "once_cell",
  "simd-adler32",
 ]
diff --git a/Cargo.toml b/Cargo.toml
index 7cd5df3a944..2ff494ab6f6 100644
--- a/Cargo.toml
+++ b/Cargo.toml
@@ -41,14 +41,15 @@ max = ["max-control", "fast", "gitoxide-core-tools-query", "gitoxide-core-tools-
 
 ## Like `max`, but only Rust is allowed.
 ##
-## This is the most compatible build as it won't need a C compiler or C toolchains to build. It's also not the fastest as or the most feature-rich in terms of available
-## transports as it uses Rust's HTTP implementation.
+## This is the most compatible build as it won't need a C compiler or C toolchains to build. Thanks to zlib-rs, you don't have to trade off between compatibility and performance.
+##
+## This uses Rust's HTTP implementation.
 ##
 ## As fast as possible, with TUI progress, progress line rendering with auto-configuration, all transports available but less mature pure Rust HTTP implementation, all `ein` tools, CLI colors and local-time support, JSON output, regex support for rev-specs.
-max-pure = ["max-control", "gix-features/zlib-rust-backend", "http-client-reqwest", "gitoxide-core-blocking-client"]
+max-pure = ["max-control", "http-client-reqwest", "gitoxide-core-blocking-client"]
 
 ## Like `max`, but with more control for configuration. See the *Package Maintainers* headline for more information.
-max-control = ["tracing", "fast-safe", "pretty-cli", "gitoxide-core-tools", "prodash-render-line", "prodash-render-tui", "prodash/render-line-autoconfigure", "gix/revparse-regex"]
+max-control = ["tracing", "fast", "pretty-cli", "gitoxide-core-tools", "prodash-render-line", "prodash-render-tui", "prodash/render-line-autoconfigure", "gix/revparse-regex"]
 
 ## All the good stuff, with less fanciness for smaller binaries.
 ##
@@ -60,7 +61,7 @@ lean = ["fast", "tracing", "pretty-cli", "http-client-curl", "gitoxide-core-tool
 ## This build is essentially limited to local operations without any fanciness.
 ##
 ## Optimized for size, no parallelism thus much slower, progress line rendering.
-small = ["pretty-cli", "gix-features/zlib-rust-backend", "prodash-render-line", "is-terminal"]
+small = ["pretty-cli", "prodash-render-line", "is-terminal"]
 
 ## Like lean, but uses Rusts async implementations for networking.
 ##
@@ -74,37 +75,20 @@ small = ["pretty-cli", "gix-features/zlib-rust-backend", "prodash-render-line",
 lean-async = ["fast", "tracing", "pretty-cli", "gitoxide-core-tools", "gitoxide-core-tools-query", "gitoxide-core-tools-corpus", "gitoxide-core-async-client", "prodash-render-line"]
 
 #! ### Package Maintainers
-#! `*-control` features leave it to you to configure C libraries, involving choices for `zlib` and transport implementation.
+#! `*-control` features leave it to you to configure C libraries, involving choices for HTTP transport implementation.
 #!
 #! Additional features *can* be provided with `--features` and are handled by the [`gix-features` crate](https://docs.rs/gix-features/latest).
-#! If nothing else is specified, the Rust implementation is used. ! Note that only one feature of each section can be enabled at a time.
-#!
-#! * **zlib**
-#!     - `gix-features/zlib-ng`
-#!     - `gix-features/zlib-ng-compat`
-#!     - `gix-features/zlib-stock`
-#!     - `gix-features/zlib-rust-backend` (*default if no choice is made*)
-#! * **HTTP** - see the *Building Blocks for mutually exclusive networking* headline
-#!
-#! #### Examples
-#!
-#! * `cargo build --release --no-default-features --features max-control,gix-features/zlib-stock,gitoxide-core-blocking-client,http-client-curl`
-#!     - Create a build just like `max`, but using the stock `zlib` library instead of `zlib-ng`
-#! * `cargo build --release --no-default-features --features max-control,http-client-reqwest,gitoxide-core-blocking-client,gix-features/zlib-ng`
-#!     - Create a build just like `max-pure`, but with faster compression due to `zlib-ng`.
+#! Note that only one HTTP transport can be enabled at a time. See the *Building Blocks for mutually exclusive networking* headline.
 
 #! ### Building Blocks
 #! Typical combinations of features of our dependencies, some of which are referred to in the `gitoxide` crate's code for conditional compilation.
 
-## Makes the crate execute as fast as possible by supporting parallel computation of otherwise long-running functions
-## as well as a faster zlib backend.
+## Makes the crate execute as fast as possible by supporting parallel computation of otherwise long-running functions.
 ## If disabled, the binary will be visibly smaller.
 fast = ["gix/max-performance", "gix/comfort"]
 
-## Makes the crate execute as fast as possible by supporting parallel computation of otherwise long-running functions
-## as well as a faster zlib backend.
-## If disabled, the binary will be visibly smaller.
-fast-safe = ["gix/max-performance-safe", "gix/comfort"]
+## Deprecated: identical to `fast`, as the fastest zlib backend is now the pure-Rust zlib-rs.
+fast-safe = ["fast"]
 
 ## Enable tracing in `gitoxide-core`.
 tracing = ["dep:tracing-forest", "dep:tracing-subscriber", "dep:tracing", "gix-features/tracing", "gix-features/tracing-detail"]
@@ -201,8 +185,8 @@ gix-ref = { opt-level = 3 }
 gix-hash = { opt-level = 3 }
 gix-actor = { opt-level = 3 }
 gix-config = { opt-level = 3 }
-miniz_oxide = { opt-level = 3 }
 sha1-checked = { opt-level = 3 }
+zlib-rs = { opt-level = 3 }
 
 [profile.release]
 overflow-checks = false
diff --git a/README.md b/README.md
index ca293fdb0fc..4b1fe4e8d19 100644
--- a/README.md
+++ b/README.md
@@ -418,8 +418,8 @@ Please take a look at the [`SHORTCOMINGS.md` file](https://github.com/GitoxideLa
 
 * **itertools** _(MIT Licensed)_
   * We use the `izip!` macro in code
-* **deflate2** _(MIT Licensed)_
-  * We use various abstractions to implement decompression and compression directly on top of the rather low-level `miniz_oxide` crate
+* **flate2** _(MIT Licensed)_
+  * We use the high-level `flate2` library to implement decompression and compression, which builds on the high-performance `zlib-rs` crate.
 
 ## 🙏 Special Thanks 🙏
 
diff --git a/etc/docker/Dockerfile.alpine b/etc/docker/Dockerfile.alpine
index 0c29bccb162..dd0bcb718b9 100644
--- a/etc/docker/Dockerfile.alpine
+++ b/etc/docker/Dockerfile.alpine
@@ -4,15 +4,13 @@ ARG GITOXIDE_VERSION=0.36.0
 FROM rust:alpine AS bootstrap_os
   # hadolint ignore=DL3018
   RUN apk upgrade --update-cache --available \
-      && apk add --no-cache --virtual .runtime-gitoxide libressl zlib-ng \
-             libressl3.8-libcrypto
+      && apk add --no-cache --virtual .runtime-gitoxide libressl libressl3.8-libcrypto
 
 
 FROM bootstrap_os AS bootstrap_build_deps
   # hadolint ignore=DL3018
   RUN apk add --no-cache --virtual .rust-builder cmake gcc musl-dev make pkgconfig \
-      && apk add --no-cache --virtual .bootstrap-gitoxide libressl-dev zlib-ng \
-             libressl3.8-libcrypto
+      && apk add --no-cache --virtual .bootstrap-gitoxide libressl-dev libressl3.8-libcrypto
 
 
 FROM bootstrap_build_deps AS bootstrap_builder
diff --git a/gix-archive/Cargo.toml b/gix-archive/Cargo.toml
index 822dfe8385d..85f19157cab 100644
--- a/gix-archive/Cargo.toml
+++ b/gix-archive/Cargo.toml
@@ -23,7 +23,8 @@ tar = ["dep:tar", "dep:gix-path"]
 tar_gz = ["tar", "dep:flate2"]
 
 ## Enable the `zip` archive format.
-zip = ["dep:zip"]
+## This also enables the `flate2` dependency in order to enable `zlib-rs` on it.
+zip = ["dep:flate2", "dep:zip"]
 
 
 [dependencies]
@@ -32,10 +33,8 @@ gix-object = { version = "^0.48.0", path = "../gix-object" }
 gix-path = { version = "^0.10.15", path = "../gix-path", optional = true }
 gix-date = { version = "^0.9.4", path = "../gix-date" }
 
-flate2 = { version = "1.1.1", optional = true }
-zip = { version = "2.6.1", optional = true, default-features = false, features = [
-    "deflate",
-] }
+flate2 = { version = "1.1.1", optional = true, default-features = false, features = ["zlib-rs"] }
+zip = { version = "2.6.1", optional = true, default-features = false, features = ["deflate"] }
 jiff = { version = "0.2.6", default-features = false, features = ["std"] }
 
 thiserror = "2.0.0"
diff --git a/gix-features/Cargo.toml b/gix-features/Cargo.toml
index d17b62f2876..0ef19601ccf 100644
--- a/gix-features/Cargo.toml
+++ b/gix-features/Cargo.toml
@@ -54,30 +54,23 @@ io-pipe = ["dep:bytes"]
 ## provide a proven and fast `crc32` implementation.
 crc32 = ["dep:crc32fast"]
 
-#! ### Mutually Exclusive ZLIB
-
-## Enable the usage of zlib related utilities to compress or decompress data.
-## The base `zlib` feature uses the `flate2` Rust crate; the other mutually exclusive features select the `flate2 backend.
-## Note that a competitive Zlib implementation is critical to `gitoxide's` object database performance.
-## Enabling this without enabling one of the other features below will use a low-performance pure-Rust backend.
-zlib = ["dep:flate2", "flate2?/rust_backend", "dep:thiserror"]
-## Use the C-based zlib-ng backend, which can compress and decompress significantly faster.
-zlib-ng = ["zlib", "flate2?/zlib-ng"]
-## Use the high-performance rust-based zlib backend on par with zlib-ng.
-## As of zlib-rs 0.5.0 (used by flate2 1.1.1), this no longer exports C symbols
-## by default, so it doesn't conflict with any other zlib library that might be
-## loaded into the same address space.
-zlib-rs = ["zlib", "flate2?/zlib-rs"]
-## Use zlib-ng via its zlib-compat API. Useful if you already need zlib for C
-## code elsewhere in your dependencies. Otherwise, use zlib-ng.
-zlib-ng-compat = ["zlib", "flate2?/zlib-ng-compat"]
-## Use a slower C-based backend which can compress and decompress significantly faster than the rust version.
-## Unlike `zlib-ng-compat`, this allows using dynamic linking with system `zlib` libraries and doesn't require cmake.
-zlib-stock = ["zlib", "flate2?/zlib"]
-## Pure Rust backend, available for completeness even though it's the default
-## if neither of the above options are set. Low performance, but pure Rust, so it
-## may build in environments where other backends don't.
-zlib-rust-backend = ["zlib", "flate2?/rust_backend"]
+## Enable the usage of zlib-related utilities to compress or decompress data.
+## This enables the `flate2` crate, and always uses the high-performance `zlib-rs` backend.
+## Note that the various past features for selecting zlib backends are now deprecated and do nothing.
+zlib = ["dep:flate2", "dep:thiserror"]
+## Deprecated: gix always uses zlib-rs.
+zlib-ng = ["zlib"]
+## Deprecated: gix always uses zlib-rs now. As of zlib-rs 0.5.0 (used by flate2
+## 1.1.1), this no longer exports C symbols # by default, so it doesn't
+## conflict with any other zlib library that might be loaded into the same
+## address space.
+zlib-rs = ["zlib"]
+## Deprecated: gix always uses zlib-rs.
+zlib-ng-compat = ["zlib"]
+## Deprecated: gix always uses zlib-rs.
+zlib-stock = ["zlib"]
+## Deprecated: gix always uses zlib-rs.
+zlib-rust-backend = ["zlib"]
 
 #! ### Other
 
@@ -128,7 +121,7 @@ bytesize = { version = "2.0.1", optional = true }
 bytes = { version = "1.0.0", optional = true }
 
 # zlib module
-flate2 = { version = "1.1.1", optional = true, default-features = false }
+flate2 = { version = "1.1.1", optional = true, default-features = false, features = ["zlib-rs"] }
 thiserror = { version = "2.0.0", optional = true }
 
 once_cell = { version = "1.21.3", optional = true }
diff --git a/gix/Cargo.toml b/gix/Cargo.toml
index d78595756f0..94bed1bfa7c 100644
--- a/gix/Cargo.toml
+++ b/gix/Cargo.toml
@@ -220,14 +220,11 @@ blocking-http-transport-reqwest-native-tls = [
 #! ZIP might not compile on all platforms, so it depends on the end-user who compiles the application to chose these based on their needs.
 
 ## Activate features that maximize performance, like using threads, but leave everything else that might affect compatibility out to allow users more fine-grained
-## control over performance features like which `zlib*` implementation to use.
+## control.
 ## No C toolchain is involved.
 max-control = ["parallel", "pack-cache-lru-static", "pack-cache-lru-dynamic"]
 
-## Activate features that maximize performance, like usage of threads, `and access to caching in object databases, skipping the ones known to cause compile failures
-## on some platforms.
-## Note that this configuration still uses a pure Rust zlib implementation which isn't the fastest compared to its C-alternatives.
-## No C toolchain is involved.
+## Deprecated: gix always uses zlib-rs, so this is equivalent to `max-performance`
 max-performance-safe = ["max-control"]
 
 ## The tempfile registry uses a better implementation of a thread-safe hashmap, relying on an external crate.
@@ -247,25 +244,20 @@ pack-cache-lru-static = ["gix-pack/pack-cache-lru-static"]
 ## Provide a hash-map based LRU cache whose eviction is based a memory cap calculated from object data.
 pack-cache-lru-dynamic = ["gix-pack/pack-cache-lru-dynamic"]
 
-## Activate other features that maximize performance, like usage of threads, `zlib-ng` and access to caching in object databases.
-## Note that some platforms might suffer from compile failures, which is when `max-performance-safe` should be used.
-max-performance = ["max-performance-safe", "zlib-ng"]
+## Enable all features required for performance. Currently, this is equivalent to `max-control`, as gix always uses zlib-rs.
+max-performance = ["max-control"]
 
-## Use the C-based zlib-ng backend, which can compress and decompress significantly faster.
-## Note that this will cause duplicate symbol errors if the application also depends on `zlib` - use `zlib-ng-compat` in that case.
-zlib-ng = ["gix-features/zlib-ng"]
+## Deprecated: gix always uses zlib-rs.
+zlib-ng = ["gix-features/zlib"]
 
-## Use the high-performance rust-based zlib backend en par with zlib-ng.
-## Note that this will cause duplicate symbol errors if the application also depends on `zlib`, without remedy.
-zlib-rs = ["gix-features/zlib-rs"]
+## Deprecated: gix always uses zlib-rs.
+zlib-rs = ["gix-features/zlib"]
 
-## Use zlib-ng via its zlib-compat API. Useful if you already need zlib for C
-## code elsewhere in your dependencies. Otherwise, use `zlib-ng`.
-zlib-ng-compat = ["gix-features/zlib-ng-compat"]
+## Deprecated: gix always uses zlib-rs.
+zlib-ng-compat = ["gix-features/zlib"]
 
-## Use a slower C-based backend which can compress and decompress significantly faster than the rust version.
-## Unlike `zlib-ng-compat`, this allows using dynamic linking with system `zlib` libraries and doesn't require cmake.
-zlib-stock = ["gix-features/zlib-stock"]
+## Deprecated: gix always uses zlib-rs.
+zlib-stock = ["gix-features/zlib"]
 
 #! #### Other
 #!
diff --git a/justfile b/justfile
index 138deaadd67..a47466ce1ee 100755
--- a/justfile
+++ b/justfile
@@ -103,10 +103,6 @@ check:
     cargo check -p gix-features --features io-pipe
     cargo check -p gix-features --features crc32
     cargo check -p gix-features --features zlib
-    cargo check -p gix-features --features zlib,zlib-ng
-    cargo check -p gix-features --features zlib,zlib-ng-compat
-    cargo check -p gix-features --features zlib-stock
-    cargo check -p gix-features --features zlib,zlib-stock
     cargo check -p gix-features --features cache-efficiency-debug
     cd gix-commitgraph; \
       set -ex; \
diff --git a/tests/journey/gix.sh b/tests/journey/gix.sh
index dd180cff8c7..2bf3f910d8f 100644
--- a/tests/journey/gix.sh
+++ b/tests/journey/gix.sh
@@ -551,15 +551,8 @@ title "gix commit-graph"
                   expect_run $WITH_FAILURE test -e "${PACK_FILE}".idx
                 }
 
-                if test "$kind" = "small" ; then
-                  suffix=miniz-oxide
-                elif test "$kind" = "max-pure"; then
-                  suffix=miniz-oxide-max
-                else
-                  suffix=zlib-ng
-                fi
                 it "creates all pack objects, but the broken ones" && {
-                  WITH_SNAPSHOT="$snapshot/broken-with-objects-dir-skip-checks-success-tree-$suffix" \
+                  WITH_SNAPSHOT="$snapshot/broken-with-objects-dir-skip-checks-success-tree" \
                   expect_run_sh $SUCCESSFULLY 'find . -type f | sort'
                 }
               )

From 633f0a86b8941ffddebfe5f36f4604925b822ab3 Mon Sep 17 00:00:00 2001
From: Sebastian Thiel <sebastian.thiel@icloud.com>
Date: Mon, 21 Apr 2025 08:46:52 +0200
Subject: [PATCH 2/2] Allow `testing::TestUrlExtension` in release builds for
 consistency (#1962)

The module is still hidden from the documentation, and those who discover
it anyway will be informed that it's not for general use.
---
 gix-url/src/lib.rs | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/gix-url/src/lib.rs b/gix-url/src/lib.rs
index caf9ef6a473..bbde5a08295 100644
--- a/gix-url/src/lib.rs
+++ b/gix-url/src/lib.rs
@@ -370,11 +370,10 @@ impl Url {
 }
 
 /// This module contains extensions to the [Url] struct which are only intended to be used
-/// for testing code. Do not use this module in production! For all intends and purposes the APIs of
+/// for testing code. Do not use this module in production! For all intents and purposes, the APIs of
 /// all functions and types exposed by this module are considered unstable and are allowed to break
 /// even in patch releases!
 #[doc(hidden)]
-#[cfg(debug_assertions)]
 pub mod testing {
     use bstr::BString;