Don't impose a default for 'address' and 'allowed' attributes #45

Merged
merged 2 commits into from Sep 8, 2014

Conversation

Projects
None yet
2 participants
Contributor

wadtech commented Dec 5, 2013

Remove the 'localhost' address and allowed attributes since they're optional in monitrc, but they don't get overwritten when setting them to null/empty in attributes json.

Collaborator

yourabi commented Dec 5, 2013

Hi @wadtech -- can you please squash this into one commit.

@wadtech wadtech set allowed and address default attributes to nil/empty, because they…
… are optional in monitrc and cannot be overridden with null/empty.
576a22d
Contributor

wadtech commented Dec 5, 2013

@yourabi All done.

@yourabi yourabi added a commit that referenced this pull request Sep 8, 2014

@yourabi yourabi Merge pull request #45 from wadtech/master
1) Don't impose a default for 'address' and 'allowed' attributes
2) Make alert blacklist configurable
68a9e24

@yourabi yourabi merged commit 68a9e24 into apsoto:master Sep 8, 2014

1 check failed

default The Travis CI build failed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment