[GitHub] ashb commented on issue #4006: [AIRFLOW-3164] Verify server certificate when connecting to LDAP

2018-11-08 Thread GitBox
ashb commented on issue #4006: [AIRFLOW-3164] Verify server certificate when connecting to LDAP URL: https://github.com/apache/incubator-airflow/pull/4006#issuecomment-437162375 @Fokko @kaxil tests are green on this now, and I've verified that it uses TLS1.2 by default so I'm happy with

[GitHub] ashb commented on issue #4006: [AIRFLOW-3164] Verify server certificate when connecting to LDAP

2018-11-08 Thread GitBox
ashb commented on issue #4006: [AIRFLOW-3164] Verify server certificate when connecting to LDAP URL: https://github.com/apache/incubator-airflow/pull/4006#issuecomment-436948799 I've rebased this (on Bolke's fork) This is

[GitHub] ashb commented on issue #4006: [AIRFLOW-3164] Verify server certificate when connecting to LDAP

2018-11-06 Thread GitBox
ashb commented on issue #4006: [AIRFLOW-3164] Verify server certificate when connecting to LDAP URL: https://github.com/apache/incubator-airflow/pull/4006#issuecomment-436214512 Fair point, they can create a custom auth backend if they want to,. I'll put that back. @bolkedebruin

[GitHub] ashb commented on issue #4006: [AIRFLOW-3164] Verify server certificate when connecting to LDAP

2018-11-05 Thread GitBox
ashb commented on issue #4006: [AIRFLOW-3164] Verify server certificate when connecting to LDAP URL: https://github.com/apache/incubator-airflow/pull/4006#issuecomment-435993735 In the fixup commit I just pushed I have re-enabled the ability to not have to use a certificate on the ldap