saji/gotelem
1
0
Fork 0
Code Issues 7 Actions Packages Releases 2 Activity

add more logging to BusEvent Broker

Browse source
This commit is contained in:
saji 2024-03-07 15:05:51 -06:00
parent 4a292aa009
commit 641c35afbd
1 changed files with 3 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
broker.go
View file

@ -37,7 +37,7 @@ func (b *Broker) Subscribe(name string) (ch chan skylab.BusEvent, err error) {
if ok { if ok {
return nil, errors.New("name already in use") return nil, errors.New("name already in use")
} }
b.logger.Info("new subscriber", "name", name) b.logger.Info("subscribe", "name", name)
ch = make(chan skylab.BusEvent, b.bufsize) ch = make(chan skylab.BusEvent, b.bufsize)
b.subs[name] = ch b.subs[name] = ch
@ -51,6 +51,7 @@ func (b *Broker) Unsubscribe(name string) {
// remove the channel from the map. We don't need to close it. // remove the channel from the map. We don't need to close it.
b.lock.Lock() b.lock.Lock()
defer b.lock.Unlock() defer b.lock.Unlock()
b.logger.Debug("unsubscribe", "name", name)
delete(b.subs, name) delete(b.subs, name)
} }
@ -59,6 +60,7 @@ func (b *Broker) Unsubscribe(name string) {
func (b *Broker) Publish(sender string, message skylab.BusEvent) { func (b *Broker) Publish(sender string, message skylab.BusEvent) {
b.lock.RLock() b.lock.RLock()
defer b.lock.RUnlock() defer b.lock.RUnlock()
b.logger.Debug("publish", "sender", sender, "message", message)
for name, ch := range b.subs { for name, ch := range b.subs {
if name == sender { if name == sender {
continue continue
@ -66,7 +68,6 @@ func (b *Broker) Publish(sender string, message skylab.BusEvent) {
// non blocking send. // non blocking send.
select { select {
case ch <- message: case ch <- message:
b.logger.Debug("sent message", "dest", name, "src", sender)
default: default:
b.logger.Warn("recipient buffer full", "dest", name) b.logger.Warn("recipient buffer full", "dest", name)
} }