Merge pull request ceph#57075 from cbodley/wip-rgw-ldap-derr

ivancich · web-flow · commit 1215afaf0def · 2024-05-09T14:32:40.000-04:00
rgw: don't print error when ldap isn't configured

Reviewed-by: J. Eric Ivancich &lt;ivancich@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`