unistack-org/micro
4
0
Fork 2
Code Issues 18 Pull Requests Actions Packages Projects Releases 331 Wiki Activity

Merge pull request #117 from unistack-org/race

Browse Source 
server: fix race in noop server
This commit was merged in pull request #117.
This commit is contained in:
Василий Толстов
2022-04-15 15:53:18 +03:00
committed by GitHub
parent f93ba9d977 74db004f51
commit b871f64ba6
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
server/noop.go
View File

@@ -277,7 +277,7 @@ func (n *noopServer) Deregister() error {
wg := sync.WaitGroup{} wg := sync.WaitGroup{}
for sb, subs := range n.subscribers { for sb, subs := range n.subscribers {
for _, sub := range subs { for idx := range subs {
if sb.Options().Context != nil { if sb.Options().Context != nil {
cx = sb.Options().Context cx = sb.Options().Context
} }
@@ -293,7 +293,7 @@ func (n *noopServer) Deregister() error {
config.Logger.Errorf(n.opts.Context, "unsubscribing from topic: %s err: %v", s.Topic(), err) config.Logger.Errorf(n.opts.Context, "unsubscribing from topic: %s err: %v", s.Topic(), err)
} }
} }
}(sub) }(subs[idx])
} }
n.subscribers[sb] = nil n.subscribers[sb] = nil
} }