set cafile when rebuilding the ssl context (#1223)

* set cafile when rebuilding the ssl context * add changelog --------- Co-authored-by: valentin-pinkau <[email protected]>
scalableminds · Nov 28, 2024 · 33d2ba6 · 33d2ba6
1 parent 4882724
commit 33d2ba6
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/webknossos/Changelog.md b/webknossos/Changelog.md
@@ -19,6 +19,7 @@ For upgrade instructions, please check the respective _Breaking Changes_ section
 ### Changed
 
 ### Fixed
+- Fixed unpickling of the SSL_Context to allow for a second or third pickling. [#1223](https://github.com/scalableminds/webknossos-libs/pull/1223)
 
 
 ## [0.16.0](https://github.com/scalableminds/webknossos-libs/releases/tag/v0.16.0) - 2024-11-27

diff --git a/webknossos/webknossos/dataset/defaults.py b/webknossos/webknossos/dataset/defaults.py
@@ -19,7 +19,7 @@ def _create_sslcontext() -> ssl.SSLContext:
 def _save_sslcontext(
     obj: ssl.SSLContext,
 ) -> tuple[Callable[[Any, Any], ssl.SSLContext], tuple[ssl._SSLMethod, Optional[str]]]:
-    cafile = getattr(obj, "cafile")
+    cafile = getattr(obj, "cafile", None)
     return _rebuild_sslcontext, (obj.protocol, cafile)
 
 
@@ -29,6 +29,7 @@ def _rebuild_sslcontext(
     ssl_context = ssl.SSLContext(protocol)
     if cafile is not None:
         ssl_context.load_verify_locations(cafile=cafile)
+        ssl_context.cafile = cafile  # type: ignore
     return ssl_context