nats-io / nats-streaming-server / 1632

92%

main: 92%

Pull #623

travis-ci

[FIXED] Possible panic and other issue with file logger

- On file log re-open, we were incorrectly call s.natsserver.ReOpenLogFile()
  since when using remote NATS Server, s.natsserver would be nil.
- Even when embedding, re-open caused a new logger to be set in the
  nats server's logging object, not the stan server.

Reproduce the logic of the NATS Server's re-opening of the log file
in Streaming Server's own logger.

Resolves #621
Resolves #622

Signed-off-by: Ivan Kozlovic <ivan@synadia.com>

23 of 23 new or added lines in 3 files covered. (100.0%)

10749 of 11704 relevant lines covered (91.84%)

47280.82 hits per line