influxdata / Litmus

testing framework
0 stars 0 forks source link

Update ldap-config to work with TLS #120

Open gshif opened 5 years ago

gshif commented 5 years ago

There is a change in master/1.7 where LDAP supports TLS connections and as such a new option is added to ldap config security = "starttls", but... since test server does not support TLS then we need to set security = "none" for ldap config that is used in tests for master and 1.7 branches.

gshif commented 5 years ago

I just made a change in sample-ldap-config.toml that lives in Litmus/scripts, but this is a temporary change (since it won't work for anything that is not 1.7 or master). Need to dynamically change the file or use a separate copy of a ldap configuration file for tests against 1.6 branch