Skip to content
This repository has been archived by the owner on Mar 9, 2022. It is now read-only.

Commit

Permalink
Merge pull request #59 from jumptrading/monitor-test-failure
Browse files Browse the repository at this point in the history
monitor: Flush after subscribing
  • Loading branch information
mjs authored Apr 3, 2018
2 parents 1b80854 + f568c8d commit 1c065ba
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions monitor/monitor.go
Original file line number Diff line number Diff line change
Expand Up @@ -50,6 +50,9 @@ func Start(conf *config.Config) (_ *Monitor, err error) {
if err != nil {
return nil, fmt.Errorf("NATS: failed to subscribe: %v", err)
}
if err := m.nc.Flush(); err != nil {
return nil, fmt.Errorf("NATS: failed to flush: %v", err)
}

m.wg.Add(1)
go m.serveHTTP()
Expand Down

0 comments on commit 1c065ba

Please sign in to comment.