diff options
author | Paweł Chmielowski <pchmielowski@process-one.net> | 2018-01-29 17:25:31 +0100 |
---|---|---|
committer | Paweł Chmielowski <pchmielowski@process-one.net> | 2018-01-29 17:25:31 +0100 |
commit | d2974cf48a1bc22c7adfedc91bafdd80c8494ee0 (patch) | |
tree | a7500fdd6ab2ef87953056dfe6779c84da36138f | |
parent | Don't return undefined from ejabberd_config:get_myhosts() (diff) |
Fix processing of ldap_memberattr_format_re option
This makes sure that "" value is handled as before options processing
overhaul.
This fixed ldap shared roster testcase
-rw-r--r-- | src/mod_shared_roster_ldap.erl | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/src/mod_shared_roster_ldap.erl b/src/mod_shared_roster_ldap.erl index 745430ca..04db34ef 100644 --- a/src/mod_shared_roster_ldap.erl +++ b/src/mod_shared_roster_ldap.erl @@ -575,7 +575,11 @@ mod_opt_type(ldap_memberattr_format) -> fun iolist_to_binary/1; mod_opt_type(ldap_memberattr_format_re) -> fun (S) -> - Re = iolist_to_binary(S), {ok, MP} = re:compile(Re), MP + Re = iolist_to_binary(S), + case Re of + <<>> -> <<>>; + _ -> {ok, MP} = re:compile(Re), MP + end end; mod_opt_type(ldap_rfilter) -> opt_type(ldap_rfilter); |