From 0c02218d0666397c6ba2b8c0097df83f83d1f50f Mon Sep 17 00:00:00 2001
From: Sadie Powell
Date: Fri, 27 Feb 2026 21:13:21 +0000
Subject: Massive rework of how internal service providers work.

- ServiceType is dead. All modules use RegisterService now.
- Modules can implement UnregisterService which is called from
  DelService now.
- Split the service type prefix into its own field.
- Renamed Service::name to Service::service_name. This revealed
  a few bugs.
---
 modules/ldapauth.cpp | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

(limited to 'modules/ldapauth.cpp')

diff --git a/modules/ldapauth.cpp b/modules/ldapauth.cpp
index ae0170ebf..41729ffeb 100644
--- a/modules/ldapauth.cpp
+++ b/modules/ldapauth.cpp
@@ -333,7 +333,7 @@ class ModuleLDAPAuth final
 public:
 	ModuleLDAPAuth()
 		: Module(VF_VENDOR, "Allows connecting users to be authenticated against an LDAP database.")
-		, LDAP(this, "LDAP")
+		, LDAP(this, "LDAPProvider")
 		, ldapAuthed(this, "ldapauth", ExtensionType::USER)
 		, ldapVhost(this, "ldapauth-vhost", ExtensionType::USER)
 	{
@@ -357,7 +357,7 @@ public:
 			{ "username", AuthField::NICKNAME },
 		});
 
-		LDAP.SetProvider("LDAP/" + tag->getString("dbid"));
+		LDAP.SetProviderName(tag->getString("dbid"));
 
 		requiredattributes.clear();
 		for (const auto& [_, rtag] : ServerInstance->Config->ConfTags("ldaprequire"))
@@ -446,7 +446,7 @@ public:
 
 		try
 		{
-			LDAP->BindAsManager(new AdminBindInterface(this, LDAP.GetProvider(), user->uuid, base, what));
+			LDAP->BindAsManager(new AdminBindInterface(this, LDAP.GetProviderName(), user->uuid, base, what));
 		}
 		catch (const LDAPException& ex)
 		{
-- 
cgit v1.3.1-10-gc9f91