diff --git a/.pylintrc b/.pylintrc deleted file mode 100644 index 530e1e3e..00000000 --- a/.pylintrc +++ /dev/null @@ -1,3 +0,0 @@ -[MESSAGES CONTROL] - disable= - django-not-available \ No newline at end of file diff --git a/modules/terraform-aws-ca-lambda/lambda_code/tls_cert/tls_cert.py b/modules/terraform-aws-ca-lambda/lambda_code/tls_cert/tls_cert.py index 4b9aaff0..4b86b921 100644 --- a/modules/terraform-aws-ca-lambda/lambda_code/tls_cert/tls_cert.py +++ b/modules/terraform-aws-ca-lambda/lambda_code/tls_cert/tls_cert.py @@ -69,7 +69,7 @@ class CaChainResponse: base64_ca_chain: str -# pylint:disable=too-many-arguments +# pylint:disable=too-many-arguments,too-many-positional-arguments def sign_tls_certificate(project, env_name, csr, ca_name, csr_info, domain, max_cert_lifetime, enable_public_crl): # get CA cert from DynamoDB ca_cert_bytes_b64 = db_list_certificates(project, env_name, ca_name)[0]["Certificate"]["B"] diff --git a/modules/terraform-aws-ca-lambda/requirements-dev.txt b/modules/terraform-aws-ca-lambda/requirements-dev.txt index 695acf91..6af14aad 100644 --- a/modules/terraform-aws-ca-lambda/requirements-dev.txt +++ b/modules/terraform-aws-ca-lambda/requirements-dev.txt @@ -1,6 +1,6 @@ bandit == 1.7.9 -black == 24.8.0 -boto3 == 1.35.29 +black == 24.10.0 +boto3 == 1.35.40 cryptography == 43.0.1 dataclasses-json == 0.6.7 prospector == 1.10.3 diff --git a/modules/terraform-aws-ca-lambda/utils/certs/ca.py b/modules/terraform-aws-ca-lambda/utils/certs/ca.py index 195a69f2..561ea393 100644 --- a/modules/terraform-aws-ca-lambda/utils/certs/ca.py +++ b/modules/terraform-aws-ca-lambda/utils/certs/ca.py @@ -51,7 +51,7 @@ def tls_cert_construct_subject_name(csr_cert, cert_request_info): return subject.x509_name() -# pylint:disable=too-many-arguments +# pylint:disable=too-many-arguments,too-many-positional-arguments def ca_kms_sign_ca_certificate_request( project, env_name, @@ -179,7 +179,7 @@ def ca_build_cert(csr_cert, ca_cert, lifetime, delta, cert_request_info): ) -# pylint:disable=too-many-arguments,too-many-locals +# pylint:disable=too-many-arguments,too-many-positional-arguments,too-many-locals def ca_kms_sign_tls_certificate_request( project, env_name, @@ -325,7 +325,7 @@ def subject_from_ca_info(ca_info, default_common_name=None): return subject -# pylint:disable=too-many-arguments +# pylint:disable=too-many-arguments,too-many-positional-arguments def ca_kms_publish_crl( ca_info, ca_key_info, diff --git a/modules/terraform-aws-ca-lambda/utils/certs/s3.py b/modules/terraform-aws-ca-lambda/utils/certs/s3.py index c82e02aa..7f27c4fd 100644 --- a/modules/terraform-aws-ca-lambda/utils/certs/s3.py +++ b/modules/terraform-aws-ca-lambda/utils/certs/s3.py @@ -13,7 +13,7 @@ def s3_download_file(bucket_name, key): Key=key, ) except client.exceptions.NoSuchKey: - print(f"file {key} not found in s3 bucket {bucket_name}") + print(f"file {key} not found in s3 bucket {bucket_name}") # noqa return None @@ -32,7 +32,7 @@ def s3_upload_file(file, bucket_name, key, content_type): print(f"uploaded {key} to s3 bucket {bucket_name}") -# pylint:disable=too-many-arguments +# pylint:disable=too-many-arguments,too-many-positional-arguments def s3_upload( external_s3_bucket_name, internal_s3_bucket_name, file, key, content_type="application/x-pkcs7-crl", external=True ): diff --git a/modules/terraform-aws-ca-lambda/utils/certs/types.py b/modules/terraform-aws-ca-lambda/utils/certs/types.py index dc157adb..019d0427 100644 --- a/modules/terraform-aws-ca-lambda/utils/certs/types.py +++ b/modules/terraform-aws-ca-lambda/utils/certs/types.py @@ -99,7 +99,7 @@ class CsrInfo: _purposes: list[str] = field(init=False, repr=False) @property - def sans(self) -> list[str]: + def sans(self) -> list[str]: # noqa: F811 if isinstance(self._sans, list): return filter_and_validate_sans(self.subject.common_name, self._sans) @@ -110,7 +110,7 @@ def sans(self, _sans: list[str]) -> None: self._sans = _sans @property - def purposes(self) -> list[str]: + def purposes(self) -> list[str]: # noqa: F811 if isinstance(self._purposes, list): return filter_and_validate_purposes(self._purposes) return ["client_auth"] diff --git a/requirements-dev.txt b/requirements-dev.txt index 37f3fee3..1ad24f12 100644 --- a/requirements-dev.txt +++ b/requirements-dev.txt @@ -2,12 +2,12 @@ asn1crypto == 1.5.1 assertpy == 1.1 bandit == 1.7.10 -black == 24.8.0 -boto3 == 1.35.29 +black == 24.10.0 +boto3 == 1.35.40 certvalidator == 0.11.1 cryptography == 43.0.1 dataclasses-json == 0.6.7 -prospector == 1.10.3 +prospector == 1.12.1 pytest == 8.3.3 requests == 2.32.3 structlog == 24.4.0 diff --git a/requirements-docs.txt b/requirements-docs.txt index 74e9a436..aefac902 100644 --- a/requirements-docs.txt +++ b/requirements-docs.txt @@ -1,4 +1,4 @@ cairosvg == 2.7.1 -mkdocs-material == 9.5.39 +mkdocs-material == 9.5.40 pillow == 10.4.0 urllib3 == 2.2.3 \ No newline at end of file diff --git a/scripts/requirements.txt b/scripts/requirements.txt index b1a96460..debe28bf 100644 --- a/scripts/requirements.txt +++ b/scripts/requirements.txt @@ -1 +1 @@ -boto3 == 1.35.29 \ No newline at end of file +boto3 == 1.35.40 \ No newline at end of file diff --git a/utils/modules/certs/crypto.py b/utils/modules/certs/crypto.py index ae2e5b40..4f4ce478 100644 --- a/utils/modules/certs/crypto.py +++ b/utils/modules/certs/crypto.py @@ -43,7 +43,7 @@ def certificate_validated(pem_cert, trust_roots, purposes=None, check_crl=True): return True -def create_csr_info( # pylint:disable=too-many-arguments +def create_csr_info( # pylint:disable=too-many-arguments,too-many-positional-arguments common_name, country=None, locality=None, diff --git a/utils/requirements.txt b/utils/requirements.txt index 37d806c3..8078d5f0 100644 --- a/utils/requirements.txt +++ b/utils/requirements.txt @@ -1,5 +1,5 @@ asn1crypto == 1.5.1 -boto3 == 1.35.29 +boto3 == 1.35.40 certvalidator == 0.11.1 cryptography == 43.0.1 validators == 0.34.0