SpiderOak / netkes

Networked key escrow server and LDAP sync code for SpiderOak Blue.
15 stars 5 forks source link

Get a 'LDAP not configured. Exiting.' status message when syncing #232

Open brian-emery opened 9 years ago

brian-emery commented 9 years ago

I have a freshly created OMVA running version 2.2.beta.2 that gets an 'LDAP not configured. Exiting.' message whenever you click the 'Sync Virtual Appliance' button. This OMVA was newly created on AWS and not upgraded from a previous version. I signed in with an enterprise account that was already created but that only had one user attached to it. I created a couple of additional local users. LDAP has never been configured for this account.

There were no obvious errors in the logs that I saw. I can give direct access to the VM if needed.

ldap_not_config

bdzim commented 9 years ago

From the screenshot it looks to be a non-ldap account. In which case LDAP is not configured. What do you expect to happen differently?

brian-emery commented 9 years ago

After some discussion, I think the status message works as is, but the button should be changed to read something like 'Sync AD/LDAP' or 'Sync with AD/LDAP' instead of 'Sync Virtual Appliance'. This will make it clear what the button's purpose is. As you said in the discussion, I think hiding the button for non-ldap accounts is also a good idea.

brian-emery commented 9 years ago

Scratch part of the last comment. It's already hidden in accounts where LDAP config is not an option. I'd still suggest changing the button text.