fix: remove deprecated py.allow_threads() calls and add pyclass(frozen, skip_from_py_object) for PyO3 0.28 compatibility

Author: bordumb

packages/auths-python/Cargo.lock

@@ -17,7 +17,7 @@ fn http_client() -> &'static reqwest::Client {
     })
 }
 
-#[pyclass]
+#[pyclass(frozen, skip_from_py_object)]
 #[derive(Clone)]
 pub struct PyArtifactPublishResult {
     #[pyo3(get)]
@@ -64,7 +64,7 @@ impl PyArtifactPublishResult {
 #[pyfunction]
 #[pyo3(signature = (attestation_json, registry_url, package_name=None))]
 pub fn publish_artifact(
-    py: Python<'_>,
+    _py: Python<'_>,
     attestation_json: String,
     registry_url: String,
     package_name: Option<String>,
@@ -77,7 +77,7 @@ pub fn publish_artifact(
         registry_url.trim_end_matches('/')
     );
 
-    py.allow_threads(move || {
+    {
         runtime().block_on(async move {
             let mut body = serde_json::json!({ "attestation": attestation });
             if let Some(ref name) = package_name {
@@ -125,5 +125,5 @@ pub fn publish_artifact(
                 }
             }
         })
-    })
+    }
 }

packages/auths-python/src/artifact_publish.rs

@@ -85,7 +85,7 @@ impl ArtifactSource for BytesArtifact {
     }
 }
 
-#[pyclass]
+#[pyclass(frozen, skip_from_py_object)]
 #[derive(Clone)]
 pub struct PyArtifactResult {
     #[pyo3(get)]
@@ -210,7 +210,7 @@ fn build_context_and_sign(
 #[pyfunction]
 #[pyo3(signature = (file_path, identity_key_alias, repo_path, passphrase=None, expires_in=None, note=None))]
 pub fn sign_artifact(
-    py: Python<'_>,
+    _py: Python<'_>,
     file_path: &str,
     identity_key_alias: &str,
     repo_path: &str,
@@ -229,9 +229,9 @@ pub fn sign_artifact(
     let alias = identity_key_alias.to_string();
     let rp = repo_path.to_string();
 
-    py.allow_threads(move || {
+    {
         build_context_and_sign(artifact, &alias, &rp, passphrase, expires_in, note)
-    })
+    }
 }
 
 /// Sign raw bytes, producing a dual-signed attestation.
@@ -251,7 +251,7 @@ pub fn sign_artifact(
 #[pyfunction]
 #[pyo3(signature = (data, identity_key_alias, repo_path, passphrase=None, expires_in=None, note=None))]
 pub fn sign_artifact_bytes(
-    py: Python<'_>,
+    _py: Python<'_>,
     data: &[u8],
     identity_key_alias: &str,
     repo_path: &str,
@@ -265,7 +265,7 @@ pub fn sign_artifact_bytes(
     let alias = identity_key_alias.to_string();
     let rp = repo_path.to_string();
 
-    py.allow_threads(move || {
+    {
         build_context_and_sign(artifact, &alias, &rp, passphrase, expires_in, note)
-    })
+    }
 }