rgw: don't print error when ldap isn't configured

cbodley · cbodley · commit 3dd8500596f0 · 2024-04-24T09:46:31.000-04:00
&gt; LDAP not started since no server URIs were provided in the configuration.

lack of ldap configuration is not an error, it's the default

Signed-off-by: Casey Bodley &lt;cbodley@redhat.com&gt;
diff --git a/src/rgw/rgw_appmain.cc b/src/rgw/rgw_appmain.cc
@@ -384,7 +384,6 @@ void rgw::AppMain::init_ldap()
   std::string ldap_bindpw = parse_rgw_ldap_bindpw(cct);
 
   if (ldap_uri.empty()) {
-    derr << "LDAP not started since no server URIs were provided in the configuration." << dendl;
     return;
   }
 

Original file line number	Diff line number	Diff line change
`@@ -384,7 +384,6 @@ void rgw::AppMain::init_ldap()`
`384`	`384`	`std::string ldap_bindpw = parse_rgw_ldap_bindpw(cct);`
`385`	`385`
`386`	`386`	`if (ldap_uri.empty()) {`
`387`		`- derr << "LDAP not started since no server URIs were provided in the configuration." << dendl;`
`388`	`387`	`return;`
`389`	`388`	`}`
`390`	`389`