Merge pull request #1699 from alphagov/pin-lxml

Pin lxml to < 5.1.1
alphagov · Apr 29, 2024 · b3d7a58 · b3d7a58
2 parents cfcad4a + d2a9074
commit b3d7a58
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/docker/ckan/2.9.9.Dockerfile b/docker/ckan/2.9.9.Dockerfile
@@ -19,7 +19,11 @@ RUN echo "pip install ckanext-datagovuk..." && \
 
     # install ckanext-datagovuk
     pip install $pipopt -U -r requirements.txt && \
-    pip install $pipopt -U -e . 
+    pip install $pipopt -U -e . && \
+
+    # pin lxml as 5.1.1 removes required property _ElementStringResult
+    # TODO: revisit for CKAN 2.10 upgrade as might not be required
+    pip install lxml<5.1.1
 
 # to run the CKAN wsgi set the WORKDIR to CKAN
 WORKDIR "$CKAN_VENV/src/ckan/"