X-Git-Url: https://gerrit.o-ran-sc.org/r/gitweb?a=blobdiff_plain;f=capifcore%2Finternal%2Fprovidermanagement%2Fprovidermanagement.go;fp=capifcore%2Finternal%2Fprovidermanagement%2Fprovidermanagement.go;h=f2781471fc1fa6e128de52833104fb4087c5f8b4;hb=3928f2d539956636d31902d9e3650a3a50410df3;hp=87f297a476eed49bee44c6c628681c0722777235;hpb=98f263137c38d750a6bb1c5d6a3a59ef21a27872;p=nonrtric%2Fplt%2Fsme.git

diff --git a/capifcore/internal/providermanagement/providermanagement.go b/capifcore/internal/providermanagement/providermanagement.go
index 87f297a..f278147 100644
--- a/capifcore/internal/providermanagement/providermanagement.go
+++ b/capifcore/internal/providermanagement/providermanagement.go
@@ -99,13 +99,7 @@ func (pm *ProviderManager) PostRegistrations(ctx echo.Context) error {
 		return sendCoreError(ctx, http.StatusBadRequest, "Provider missing required ApiProvDomInfo")
 	}
 
-	pm.lock.Lock()
-	defer pm.lock.Unlock()
-
-	newProvider.ApiProvDomId = pm.getDomainId(newProvider.ApiProvDomInfo)
-
-	pm.registerFunctions(newProvider.ApiProvFuncs)
-	pm.onboardedProviders[*newProvider.ApiProvDomId] = newProvider
+	pm.prepareNewProvider(&newProvider)
 
 	uri := ctx.Request().Host + ctx.Request().URL.String()
 	ctx.Response().Header().Set(echo.HeaderLocation, ctx.Scheme()+`://`+path.Join(uri, *newProvider.ApiProvDomId))
@@ -118,14 +112,24 @@ func (pm *ProviderManager) PostRegistrations(ctx echo.Context) error {
 	return nil
 }
 
-func (pm *ProviderManager) DeleteRegistrationsRegistrationId(ctx echo.Context, registrationId string) error {
+func (pm *ProviderManager) prepareNewProvider(newProvider *provapi.APIProviderEnrolmentDetails) {
 	pm.lock.Lock()
 	defer pm.lock.Unlock()
 
+	newProvider.ApiProvDomId = pm.getDomainId(newProvider.ApiProvDomInfo)
+
+	pm.registerFunctions(newProvider.ApiProvFuncs)
+	pm.onboardedProviders[*newProvider.ApiProvDomId] = *newProvider
+}
+
+func (pm *ProviderManager) DeleteRegistrationsRegistrationId(ctx echo.Context, registrationId string) error {
+
 	log.Debug(pm.onboardedProviders)
 	if _, ok := pm.onboardedProviders[registrationId]; ok {
 		log.Debug("Deleting provider", registrationId)
+		pm.lock.Lock()
 		delete(pm.onboardedProviders, registrationId)
+		pm.lock.Unlock()
 	}
 
 	return ctx.NoContent(http.StatusNoContent)