fixed dispatcher() on cutoff conditions

[kristinapathak]

fixes #208

[codecov]

Codecov Report

Merging #210 into master will increase coverage by 0.40%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #210      +/-   ##
==========================================
+ Coverage   69.27%   69.67%   +0.40%     
==========================================
  Files           7        7              
  Lines         742      742              
==========================================
+ Hits          514      517       +3     
+ Misses        216      214       -2     
+ Partials       12       11       -1     

Impacted Files	Coverage Δ
outboundSender.go	84.31% <0.00%> (+0.98%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0723afd...603c5f1. Read the comment docs.

[schmidtw]

Remove the Loop: label.

[kristinapathak]

We need Loop: for this line though:

caduceus/outboundSender.go

Line 470 in b887423

break Loop

Otherwise, we would just break out of the select statement. Is there a better way to do this?

[johnabass]

This works, but do we still need the Loop label at all?

Quick thought: Seems like this code never exits now, which isn't much of an issue if we never intend to quiesce/restart it.

[gargidb]

I think we still need the Loop label to break out if the queue is empty.