airflow-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Marcelo G. Almiron (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (AIRFLOW-927) LDAP authtorization: ldap3 doesn't understand anything coming from configuration file
Date Wed, 01 Mar 2017 11:52:46 GMT

     [ https://issues.apache.org/jira/browse/AIRFLOW-927?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Marcelo G. Almiron updated AIRFLOW-927:
---------------------------------------
    Description: 
Function `get_ldap_connection` on airflow/contrib/auth/backends/ldap_auth.py does not understand
'future.types.newstr.newstr' objects coming from `configuration.get("ldap", "bind_user")`
and `configuration.get("ldap", "bind_password")`.

The same can be observed in the `group_contains_user` function. Here the `search` function
does not consider the arguments `configuration.get("ldap", "basedn")`, `configuration.get("ldap",
"superuser_filter")`, not `configuration.get("ldap", "user_name_attr")` as expected. 

A quick fix would be to convert all these values to `str` as soon as they are read from the
configuration file, but this is not a definitive fix.

  was:Function `get_ldap_connection` on airflow/contrib/auth/backends/ldap_auth.py does not
understand 'future.types.newstr.newstr' objects coming from `configuration.get("ldap", "bind_user")`
and `configuration.get("ldap", "bind_password")`.

        Summary: LDAP authtorization: ldap3 doesn't understand anything coming from configuration
file  (was: LDAP authtorization: ldap3 doesn't understand credentials from configuration file)

> LDAP authtorization: ldap3 doesn't understand anything coming from configuration file
> -------------------------------------------------------------------------------------
>
>                 Key: AIRFLOW-927
>                 URL: https://issues.apache.org/jira/browse/AIRFLOW-927
>             Project: Apache Airflow
>          Issue Type: Bug
>          Components: contrib
>    Affects Versions: Airflow 1.7.1
>            Reporter: Marcelo G. Almiron
>
> Function `get_ldap_connection` on airflow/contrib/auth/backends/ldap_auth.py does not
understand 'future.types.newstr.newstr' objects coming from `configuration.get("ldap", "bind_user")`
and `configuration.get("ldap", "bind_password")`.
> The same can be observed in the `group_contains_user` function. Here the `search` function
does not consider the arguments `configuration.get("ldap", "basedn")`, `configuration.get("ldap",
"superuser_filter")`, not `configuration.get("ldap", "user_name_attr")` as expected. 
> A quick fix would be to convert all these values to `str` as soon as they are read from
the configuration file, but this is not a definitive fix.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Mime
View raw message