cmd/scollector: Check for NaN in Valid() #1864

Merged
merged 1 commit into from Aug 12, 2016

Conversation

Projects
None yet
2 participants
@mathpl
Contributor

mathpl commented Aug 12, 2016

Small edge case I ran into. ParseFloat() can succeed and return a NaN which then cannot be marshaled into json later on.

@captncraig captncraig merged commit 8fb59ef into bosun-monitor:master Aug 12, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment