From 6500714a93c6658749df8e477a729d9053f5c9b1 Mon Sep 17 00:00:00 2001 From: Faye Duxovni Date: Mon, 3 Apr 2023 17:27:00 -0400 Subject: Don't treat nil as an unknown type of error (zulip) (#2024) Fixes #1869 --- bridge/zulip/zulip.go | 45 +++++++++++++++++++++++---------------------- 1 file changed, 23 insertions(+), 22 deletions(-) diff --git a/bridge/zulip/zulip.go b/bridge/zulip/zulip.go index c912b6f0..9b8b73c2 100644 --- a/bridge/zulip/zulip.go +++ b/bridge/zulip/zulip.go @@ -106,30 +106,31 @@ func (b *Bzulip) getChannel(id int) string { func (b *Bzulip) handleQueue() error { for { messages, err := b.q.GetEvents() - switch err { - case gzb.BackoffError: - time.Sleep(time.Second * 5) - case gzb.NoJSONError: - b.Log.Error("Response wasn't JSON, server down or restarting? sleeping 10 seconds") - time.Sleep(time.Second * 10) - case gzb.BadEventQueueError: - b.Log.Info("got a bad event queue id error, reconnecting") - b.bot.Queues = nil - for { - b.q, err = b.bot.RegisterAll() - if err != nil { - b.Log.Errorf("reconnecting failed: %s. Sleeping 10 seconds", err) - time.Sleep(time.Second * 10) + if err != nil { + switch err { + case gzb.BackoffError: + time.Sleep(time.Second * 5) + case gzb.NoJSONError: + b.Log.Error("Response wasn't JSON, server down or restarting? sleeping 10 seconds") + time.Sleep(time.Second * 10) + case gzb.BadEventQueueError: + b.Log.Info("got a bad event queue id error, reconnecting") + b.bot.Queues = nil + for { + b.q, err = b.bot.RegisterAll() + if err != nil { + b.Log.Errorf("reconnecting failed: %s. Sleeping 10 seconds", err) + time.Sleep(time.Second * 10) + } + break } - break + case gzb.HeartbeatError: + b.Log.Debug("heartbeat received.") + default: + b.Log.Debugf("receiving error: %#v", err) + time.Sleep(time.Second * 10) } - case gzb.HeartbeatError: - b.Log.Debug("heartbeat received.") - default: - b.Log.Debugf("receiving error: %#v", err) - time.Sleep(time.Second * 10) - } - if err != nil { + continue } for _, m := range messages { -- cgit v1.2.3