Revert "Uncomment cryptography dependency in Python build test fixtures (#8667)" (#8668)

roger-zhangg · web-flow · commit 216c71f2d143 · 2026-02-20T03:20:23.000Z
This reverts commit 8792a04.
diff --git a/tests/integration/buildcmd/build_integ_base.py b/tests/integration/buildcmd/build_integ_base.py
@@ -747,7 +747,7 @@ class BuildIntegPythonBase(BuildIntegBase):
         "__init__.py",
         "main.py",
         "numpy",
-        "cryptography",
+        # 'cryptography',
     }
 
     FUNCTION_LOGICAL_ID = "Function"
diff --git a/tests/integration/buildcmd/test_build_cmd.py b/tests/integration/buildcmd/test_build_cmd.py
@@ -398,7 +398,7 @@ class TestBuildCommand_SingleFunctionBuilds(BuildIntegBase):
         "__init__.py",
         "main.py",
         "numpy",
-        "cryptography",
+        # 'cryptography',
         "requirements.txt",
     }
 
@@ -1753,7 +1753,7 @@ class TestBuildWithNestedStacksImage(NestedBuildIntegBase):
         "__init__.py",
         "main.py",
         "numpy",
-        "cryptography",
+        # 'cryptography',
         "requirements.txt",
     }
 
@@ -1927,7 +1927,7 @@ class TestBuildWithS3FunctionsOrLayers(NestedBuildIntegBase):
         "__init__.py",
         "main.py",
         "numpy",
-        "cryptography",
+        # 'cryptography',
         "requirements.txt",
     }
 
diff --git a/tests/integration/testdata/buildcmd/PyLayer/requirements.txt b/tests/integration/testdata/buildcmd/PyLayer/requirements.txt
@@ -3,4 +3,6 @@
 # NOTE: Fixing to <1.20.3 as numpy1.20.3 started to use a new wheel naming convention (PEP 600)
 numpy<1.20.3; python_version < '3.10'
 numpy==1.26.4; python_version >= '3.10'
-cryptography~=42.0
+# `cryptography` has a dependency on `pycparser` which, for some reason doesn't build inside a Docker container.
+# Turning this off until we resolve this issue: https://github.com/awslabs/aws-lambda-builders/issues/29
+# cryptography~=2.4
diff --git a/tests/integration/testdata/buildcmd/PyLayerMake/requirements.txt b/tests/integration/testdata/buildcmd/PyLayerMake/requirements.txt
@@ -3,4 +3,6 @@
 # NOTE: Fixing to <1.20.3 as numpy1.20.3 started to use a new wheel naming convention (PEP 600)
 numpy<1.20.3; python_version < '3.10'
 numpy==1.26.4; python_version >= '3.10'
-cryptography~=42.0
+# `cryptography` has a dependency on `pycparser` which, for some reason doesn't build inside a Docker container.
+# Turning this off until we resolve this issue: https://github.com/awslabs/aws-lambda-builders/issues/29
+# cryptography~=2.4
diff --git a/tests/integration/testdata/buildcmd/Python/main.py b/tests/integration/testdata/buildcmd/Python/main.py
@@ -1,11 +1,12 @@
 import numpy
 
-from cryptography.fernet import Fernet
+
+# from cryptography.fernet import Fernet
 
 
 def handler(event, context):
     # Try using some of the modules to make sure they work & don't crash the process
-    print(Fernet.generate_key())
+    # print(Fernet.generate_key())
 
     return {"pi": "{0:.2f}".format(numpy.pi)}
 
diff --git a/tests/integration/testdata/buildcmd/Python/requirements.txt b/tests/integration/testdata/buildcmd/Python/requirements.txt
@@ -4,4 +4,6 @@
 numpy<1.20.3; python_version <= '3.9'
 numpy==2.1.3; python_version >= '3.10' and python_version < '3.14'
 numpy==2.3.4; python_version >= '3.14'
-cryptography~=42.0
+# `cryptography` has a dependency on `pycparser` which, for some reason doesn't build inside a Docker container.
+# Turning this off until we resolve this issue: https://github.com/awslabs/aws-lambda-builders/issues/29
+# cryptography~=2.4
diff --git a/tests/integration/testdata/buildcmd/PythonContainerEnvVars/requirements.txt b/tests/integration/testdata/buildcmd/PythonContainerEnvVars/requirements.txt
@@ -1,7 +0,0 @@
-# These are some hard packages to build. Using them here helps us verify that building works on various platforms
-
-# NOTE: Fixing to <1.20.3 as numpy1.20.3 started to use a new wheel naming convention (PEP 600)
-numpy<1.20.3; python_version <= '3.9'
-numpy==2.1.3; python_version >= '3.10' and python_version < '3.14'
-numpy==2.3.4; python_version >= '3.14'
-cryptography~=42.0
diff --git a/tests/integration/testdata/buildcmd/PythonImage/requirements.txt b/tests/integration/testdata/buildcmd/PythonImage/requirements.txt
@@ -4,4 +4,6 @@
 numpy<1.20.3; python_version < '3.10'
 numpy==2.1.3; python_version >= '3.10' and python_version < '3.14'
 numpy==2.3.4; python_version >= '3.14'
-cryptography~=42.0
+# `cryptography` has a dependency on `pycparser` which, for some reason doesn't build inside a Docker container.
+# Turning this off until we resolve this issue: https://github.com/awslabs/aws-lambda-builders/issues/29
+# cryptography~=2.4
diff --git a/tests/integration/testdata/buildcmd/PythonImagesWithSharedCode/requirements.txt b/tests/integration/testdata/buildcmd/PythonImagesWithSharedCode/requirements.txt
@@ -4,4 +4,6 @@
 numpy<1.20.3; python_version < '3.10'
 numpy==2.1.3; python_version >= '3.10' and python_version < '3.14'
 numpy==2.3.4; python_version >= '3.14'
-cryptography~=42.0
+# `cryptography` has a dependency on `pycparser` which, for some reason doesn't build inside a Docker container.
+# Turning this off until we resolve this issue: https://github.com/awslabs/aws-lambda-builders/issues/29
+# cryptography~=2.4
diff --git a/tests/integration/testdata/buildcmd/PythonPEP600/requirements.txt b/tests/integration/testdata/buildcmd/PythonPEP600/requirements.txt
@@ -7,4 +7,3 @@ numpy==2.3.4; python_version >= '3.14'
 greenlet==3.1.1; python_version < '3.14'
 greenlet==3.2.4; python_version >= '3.14'
 sqlalchemy==2.0.36
-cryptography~=42.0
diff --git a/tests/integration/testdata/buildcmd/PythonParentPackages/src/fnone/requirements.txt b/tests/integration/testdata/buildcmd/PythonParentPackages/src/fnone/requirements.txt
@@ -3,4 +3,6 @@
 # NOTE: Fixing to <1.20.3 as numpy1.20.3 started to use a new wheel naming convention (PEP 600)
 numpy<1.20.3; python_version <= '3.9'
 numpy==2.1.3; python_version >= '3.10'
-cryptography~=42.0
+# `cryptography` has a dependency on `pycparser` which, for some reason doesn't build inside a Docker container.
+# Turning this off until we resolve this issue: https://github.com/awslabs/aws-lambda-builders/issues/29
+# cryptography~=2.4
diff --git a/tests/integration/testdata/buildcmd/PythonParentPackages/src/fntwo/requirements.txt b/tests/integration/testdata/buildcmd/PythonParentPackages/src/fntwo/requirements.txt
@@ -3,4 +3,6 @@
 # NOTE: Fixing to <1.20.3 as numpy1.20.3 started to use a new wheel naming convention (PEP 600)
 numpy<1.20.3; python_version <= '3.9'
 numpy==2.1.3; python_version >= '3.10'
-cryptography~=42.0
+# `cryptography` has a dependency on `pycparser` which, for some reason doesn't build inside a Docker container.
+# Turning this off until we resolve this issue: https://github.com/awslabs/aws-lambda-builders/issues/29
+# cryptography~=2.4
diff --git a/tests/integration/testdata/buildcmd/PythonPyProject/main.py b/tests/integration/testdata/buildcmd/PythonPyProject/main.py
@@ -1,11 +1,12 @@
 import numpy
 
-from cryptography.fernet import Fernet
+
+# from cryptography.fernet import Fernet
 
 
 def handler(event, context):
     # Try using some of the modules to make sure they work & don't crash the process
-    print(Fernet.generate_key())
+    # print(Fernet.generate_key())
 
     return {"pi": "{0:.2f}".format(numpy.pi)}
 
diff --git a/tests/integration/testdata/buildcmd/PythonPyProject/pyproject.toml b/tests/integration/testdata/buildcmd/PythonPyProject/pyproject.toml
@@ -5,5 +5,7 @@ dependencies = [
   "numpy==2.0.0; python_version <= '3.9'",
   "numpy==2.1.3; python_version >= '3.10' and python_version < '3.14'",
   "numpy==2.3.4; python_version >= '3.14'",
-  "cryptography~=42.0",
+  # `cryptography` has a dependency on `pycparser` which, for some reason doesn't build inside a Docker container.
+  # Turning this off until we resolve this issue: https://github.com/awslabs/aws-lambda-builders/issues/29
+  # "cryptography~=2.4"
 ]
diff --git a/tests/integration/testdata/buildcmd/PythonWithLayer/requirements.txt b/tests/integration/testdata/buildcmd/PythonWithLayer/requirements.txt
@@ -1,3 +1,5 @@
 # These are some hard packages to build. Using them here helps us verify that building works on various platforms
 
-cryptography~=42.0
+# `cryptography` has a dependency on `pycparser` which, for some reason doesn't build inside a Docker container.
+# Turning this off until we resolve this issue: https://github.com/awslabs/aws-lambda-builders/issues/29
+# cryptography~=2.4
diff --git a/tests/integration/testdata/buildcmd/terraform/zip_based_lambda_functions_local_backend/my_layer_code/requirements.txt b/tests/integration/testdata/buildcmd/terraform/zip_based_lambda_functions_local_backend/my_layer_code/requirements.txt
@@ -3,4 +3,6 @@
 # NOTE: Fixing to <1.20.3 as numpy1.20.3 started to use a new wheel naming convention (PEP 600)
 numpy<1.20.3; python_version < '3.10'
 numpy==1.26.4; python_version >= '3.10'
-cryptography~=42.0
+# `cryptography` has a dependency on `pycparser` which, for some reason doesn't build inside a Docker container.
+# Turning this off until we resolve this issue: https://github.com/awslabs/aws-lambda-builders/issues/29
+# cryptography~=2.4
diff --git a/tests/integration/testdata/buildcmd/terraform/zip_based_lambda_functions_local_backend_container_windows/my_layer_code/requirements.txt b/tests/integration/testdata/buildcmd/terraform/zip_based_lambda_functions_local_backend_container_windows/my_layer_code/requirements.txt
@@ -3,4 +3,6 @@
 # NOTE: Fixing to <1.20.3 as numpy1.20.3 started to use a new wheel naming convention (PEP 600)
 numpy<1.20.3; python_version < '3.10'
 numpy==1.26.4; python_version >= '3.10'
-cryptography~=42.0
+# `cryptography` has a dependency on `pycparser` which, for some reason doesn't build inside a Docker container.
+# Turning this off until we resolve this issue: https://github.com/awslabs/aws-lambda-builders/issues/29
+# cryptography~=2.4
diff --git a/tests/integration/testdata/buildcmd/terraform/zip_based_lambda_functions_local_backend_windows/my_layer_code/requirements.txt b/tests/integration/testdata/buildcmd/terraform/zip_based_lambda_functions_local_backend_windows/my_layer_code/requirements.txt
@@ -3,4 +3,6 @@
 # NOTE: Fixing to <1.20.3 as numpy1.20.3 started to use a new wheel naming convention (PEP 600)
 numpy<1.20.3; python_version < '3.10'
 numpy==1.26.4; python_version >= '3.10'
-cryptography~=42.0
+# `cryptography` has a dependency on `pycparser` which, for some reason doesn't build inside a Docker container.
+# Turning this off until we resolve this issue: https://github.com/awslabs/aws-lambda-builders/issues/29
+# cryptography~=2.4
diff --git a/tests/integration/testdata/buildcmd/terraform/zip_based_lambda_functions_s3_backend/my_layer_code/requirements.txt b/tests/integration/testdata/buildcmd/terraform/zip_based_lambda_functions_s3_backend/my_layer_code/requirements.txt
@@ -3,4 +3,6 @@
 # NOTE: Fixing to <1.20.3 as numpy1.20.3 started to use a new wheel naming convention (PEP 600)
 numpy<1.20.3; python_version < '3.10'
 numpy==1.26.4; python_version >= '3.10'
-cryptography~=42.0
+# `cryptography` has a dependency on `pycparser` which, for some reason doesn't build inside a Docker container.
+# Turning this off until we resolve this issue: https://github.com/awslabs/aws-lambda-builders/issues/29
+# cryptography~=2.4
diff --git a/tests/integration/testdata/buildcmd/terraform/zip_based_lambda_functions_s3_backend_container_windows/my_layer_code/requirements.txt b/tests/integration/testdata/buildcmd/terraform/zip_based_lambda_functions_s3_backend_container_windows/my_layer_code/requirements.txt
@@ -2,4 +2,6 @@
 
 # NOTE: Fixing to <1.20.3 as numpy1.20.3 started to use a new wheel naming convention (PEP 600)
 numpy<1.20.3
-cryptography~=42.0
+# `cryptography` has a dependency on `pycparser` which, for some reason doesn't build inside a Docker container.
+# Turning this off until we resolve this issue: https://github.com/awslabs/aws-lambda-builders/issues/29
+# cryptography~=2.4
diff --git a/tests/integration/testdata/buildcmd/terraform/zip_based_lambda_functions_s3_backend_windows/my_layer_code/requirements.txt b/tests/integration/testdata/buildcmd/terraform/zip_based_lambda_functions_s3_backend_windows/my_layer_code/requirements.txt
@@ -3,4 +3,6 @@
 # NOTE: Fixing to <1.20.3 as numpy1.20.3 started to use a new wheel naming convention (PEP 600)
 numpy<1.20.3; python_version < '3.10'
 numpy==1.26.4; python_version >= '3.10'
-cryptography~=42.0
+# `cryptography` has a dependency on `pycparser` which, for some reason doesn't build inside a Docker container.
+# Turning this off until we resolve this issue: https://github.com/awslabs/aws-lambda-builders/issues/29
+# cryptography~=2.4

Original file line number	Diff line number	Diff line change
`@@ -747,7 +747,7 @@ class BuildIntegPythonBase(BuildIntegBase):`
`747`	`747`	`"__init__.py",`
`748`	`748`	`"main.py",`
`749`	`749`	`"numpy",`
`750`		`- "cryptography",`
	`750`	`+ # 'cryptography',`
`751`	`751`	`}`
`752`	`752`
`753`	`753`	`FUNCTION_LOGICAL_ID = "Function"`