Release 5.0.9 (#20)

* 🔧 update CI to drop non official Python setup

CPython 3.13t available without workaround

* ⚡ release the GIL in hpack encode/decode

for the optimized build only

* 🔖 bump version to 5.0.9

* 📝 write changelog for 5.0.9

* 🎨 reformat file lib.rs

* 🔧 fix 3.13t x86 build

Author: Ousret

.github/workflows/CI.yml

@@ -138,13 +138,8 @@ jobs:
     steps:
       - uses: actions/checkout@v4
       - uses: actions/setup-python@v5
-        if: matrix.python_version != '3.13t'
         with:
           python-version: ${{ matrix.python_version }}
-      - uses: Quansight-Labs/setup-python@v5
-        if: matrix.python_version == '3.13t'
-        with:
-          python-version: 3.13t
       - name: Build wheels
         uses: PyO3/maturin-action@v1
         env:
@@ -193,23 +188,16 @@ jobs:
             python_version: 'pypy-3.10'
           - target: x86
             python_version: 'pypy-3.11'
-          - target: x86
-            python_version: '3.13t'
     steps:
       - uses: actions/checkout@v4
       - uses: actions/setup-python@v5
-        if: matrix.python_version != '3.13t'
+        if: matrix.target != 'aarch64'  # solely rely on python3-ddl generator for aarch64
         with:
           python-version: ${{ matrix.python_version }}
-          architecture: ${{ matrix.target == 'x86' && 'x86' || 'x64' }}
-      - uses: Quansight-Labs/setup-python@v5
-        if: matrix.python_version == '3.13t' && matrix.target == 'x64'
-        with:
-          python-version: '3.13t'
-          architecture: x64
+          architecture: ${{ matrix.target }}
       - name: Build wheels (normal)
         uses: PyO3/maturin-action@v1
-        if: matrix.python_version != '3.13t' || matrix.target == 'x64'
+        if: matrix.python_version != '3.13t' || matrix.target != 'aarch64'
         env:
           UNSAFE_PYO3_SKIP_VERSION_CHECK: 1
         with:
@@ -245,13 +233,8 @@ jobs:
     steps:
       - uses: actions/checkout@v4
       - uses: actions/setup-python@v5
-        if: matrix.python_version != '3.13t'
         with:
           python-version: ${{ matrix.python_version }}
-      - uses: Quansight-Labs/setup-python@v5
-        if: matrix.python_version == '3.13t'
-        with:
-          python-version: 3.13t
       - name: Build wheels
         uses: PyO3/maturin-action@v1
         env:

CHANGELOG.rst

@@ -1,6 +1,12 @@
 Release History
 ===============
 
+5.0.9 (2025-04-21)
+------------------
+
+- Release the GIL during hpack encoding/decoding in the optimized build.
+- (rust) Upgrade pyo3 to 0.24.1
+
 5.0.8 (2025-03-06)
 ------------------
 

Cargo.lock

@@ -1,6 +1,6 @@
 [package]
 name = "jh2"
-version = "5.0.8"
+version = "5.0.9"
 edition = "2021"
 
 # See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html

Cargo.toml

@@ -7,4 +7,4 @@
 
 from __future__ import annotations
 
-__version__ = "5.0.8"
+__version__ = "5.0.9"

jh2/__init__.py

@@ -30,7 +30,7 @@ impl Encoder {
     pub fn encode<'a>(
         &mut self,
         py: Python<'a>,
-        headers: Vec<(Bound<'_, PyBytes>, Bound<'_, PyBytes>, bool)>,
+        headers: Vec<(Vec<u8>, Vec<u8>, bool)>,
         huffman: Option<bool>,
     ) -> PyResult<Bound<'a, PyBytes>> {
         let mut flags = InternalEncoder::BEST_FORMAT;
@@ -41,27 +41,29 @@ impl Encoder {
 
         let mut dst = Vec::new();
 
-        for (header, value, sensitive) in headers.iter() {
-            let mut header_flags: u8 = flags;
+        let encode_res = py.allow_threads(|| {
+            for (header, value, sensitive) in headers.iter() {
+                let mut header_flags: u8 = flags;
 
-            if *sensitive {
-                header_flags |= InternalEncoder::NEVER_INDEXED;
-            } else {
-                header_flags |= InternalEncoder::WITH_INDEXING;
-            }
+                if *sensitive {
+                    header_flags |= InternalEncoder::NEVER_INDEXED;
+                } else {
+                    header_flags |= InternalEncoder::WITH_INDEXING;
+                }
 
-            let enc_res = self.inner.encode(
-                (
-                    header.as_bytes().to_vec(),
-                    value.as_bytes().to_vec(),
-                    header_flags,
-                ),
-                &mut dst,
-            );
+                let enc_res = self
+                    .inner
+                    .encode((header.to_vec(), value.to_vec(), header_flags), &mut dst);
 
-            if enc_res.is_err() {
-                return Err(HPACKError::new_err("operation failed"));
+                if enc_res.is_err() {
+                    return Err(HPACKError::new_err("operation failed"));
+                }
             }
+            Ok(())
+        });
+
+        if encode_res.is_err() {
+            return Err(encode_res.err().unwrap());
         }
 
         Ok(PyBytes::new(py, dst.as_slice()))
@@ -71,7 +73,7 @@ impl Encoder {
     pub fn add<'a>(
         &mut self,
         py: Python<'a>,
-        header: (Bound<'_, PyBytes>, Bound<'_, PyBytes>),
+        header: (Vec<u8>, Vec<u8>),
         sensitive: bool,
         huffman: Option<bool>,
     ) -> PyResult<Bound<'a, PyBytes>> {
@@ -89,14 +91,10 @@ impl Encoder {
 
         let mut dst = Vec::new();
 
-        let enc_res = self.inner.encode(
-            (
-                header.0.as_bytes().to_vec(),
-                header.1.as_bytes().to_vec(),
-                flags,
-            ),
-            &mut dst,
-        );
+        let enc_res = py.allow_threads(|| {
+            self.inner
+                .encode((header.0.to_vec(), header.1.to_vec(), flags), &mut dst)
+        });
 
         if enc_res.is_err() {
             return Err(HPACKError::new_err("operation failed"));
@@ -159,7 +157,7 @@ impl Decoder {
 
             let mut data = Vec::with_capacity(1);
 
-            let dec_res = self.inner.decode_exact(&mut buf, &mut data);
+            let dec_res = py.allow_threads(|| self.inner.decode_exact(&mut buf, &mut data));
 
             if dec_res.is_err() {
                 return Err(HPACKError::new_err("operation failed"));