uppal/cgrates
1
0
Fork 1
mirror of https://github.com/cgrates/cgrates.git synced 2026-02-11 10:06:24 +05:00
Code Issues Packages Projects Releases Wiki Activity

add guardian lock for dispatcher route race condition

Browse Source
This commit is contained in:
ionutboangiu
2026-01-27 18:21:17 +02:00
committed by Dan Christian Bogos
parent ada413c856
commit 65a7f1c145
1 changed files with 8 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
dispatchers/libdispatcher.go
View File

@@ -24,7 +24,9 @@ import (
"strconv" "strconv"
"sync" "sync"
"github.com/cgrates/cgrates/config"
"github.com/cgrates/cgrates/engine" "github.com/cgrates/cgrates/engine"
"github.com/cgrates/cgrates/guardian"
"github.com/cgrates/cgrates/utils" "github.com/cgrates/cgrates/utils"
) )
@@ -230,6 +232,9 @@ func (*singleResultstrategyDispatcher) dispatch(dm *engine.DataManager, routeID
if routeID != nil && *routeID != "" { if routeID != nil && *routeID != "" {
// overwrite routeID with RouteID:Subsystem // overwrite routeID with RouteID:Subsystem
*routeID = utils.ConcatenatedKey(*routeID, subsystem) *routeID = utils.ConcatenatedKey(*routeID, subsystem)
guardID := utils.ConcatenatedKey(utils.DispatcherS, utils.RouteID, *routeID)
refID := guardian.Guardian.GuardIDs("", config.CgrConfig().GeneralCfg().LockingTimeout, guardID)
defer guardian.Guardian.UnguardIDs(refID)
// use previously discovered route // use previously discovered route
if x, ok := engine.Cache.Get(utils.CacheDispatcherRoutes, if x, ok := engine.Cache.Get(utils.CacheDispatcherRoutes,
*routeID); ok && x != nil { *routeID); ok && x != nil {
@@ -316,6 +321,9 @@ func (ld *loadStrategyDispatcher) dispatch(dm *engine.DataManager, routeID *stri
if routeID != nil && *routeID != "" { if routeID != nil && *routeID != "" {
// overwrite routeID with RouteID:Subsystem // overwrite routeID with RouteID:Subsystem
*routeID = utils.ConcatenatedKey(*routeID, subsystem) *routeID = utils.ConcatenatedKey(*routeID, subsystem)
guardID := utils.ConcatenatedKey(utils.DispatcherS, utils.RouteID, *routeID)
refID := guardian.Guardian.GuardIDs("", config.CgrConfig().GeneralCfg().LockingTimeout, guardID)
defer guardian.Guardian.UnguardIDs(refID)
// use previously discovered route // use previously discovered route
if x, ok := engine.Cache.Get(utils.CacheDispatcherRoutes, if x, ok := engine.Cache.Get(utils.CacheDispatcherRoutes,
*routeID); ok && x != nil { *routeID); ok && x != nil {