[GitHub] [airflow] hussein-awala commented on a diff in pull request #30310: In `DockerOperator`, adding an attribute `tls_verify` to choose whether to validate certificate (#30309)

2023-04-13 Thread via GitHub
hussein-awala commented on code in PR #30310: URL: https://github.com/apache/airflow/pull/30310#discussion_r1166036989 ## tests/providers/docker/operators/test_docker.py: ## @@ -65,6 +66,7 @@ def test_hook_usage(docker_hook_patcher, docker_conn_id, tls_params: dict): "

[GitHub] [airflow] hussein-awala commented on a diff in pull request #30310: In `DockerOperator`, adding an attribute `tls_verify` to choose whether to validate certificate (#30309)

2023-04-01 Thread via GitHub
hussein-awala commented on code in PR #30310: URL: https://github.com/apache/airflow/pull/30310#discussion_r1155216890 ## airflow/providers/docker/operators/docker.py: ## @@ -186,6 +187,7 @@ def __init__( tls_ca_cert: str | None = None, tls_client_cert: str | N

[GitHub] [airflow] hussein-awala commented on a diff in pull request #30310: In `DockerOperator`, adding an attribute `tls_verify` to choose whether to validate certificate (#30309)

2023-03-26 Thread via GitHub
hussein-awala commented on code in PR #30310: URL: https://github.com/apache/airflow/pull/30310#discussion_r1148671668 ## airflow/providers/docker/hooks/docker.py: ## @@ -87,6 +87,7 @@ def construct_tls_config( ca_cert: str | None = None, client_cert: str | Non