airbnb / streamalert / 5307

95%

push

travis-ci

Merges 2.2.0 (#929)

* Adds Alert Publisher framework (#900)

* Squashed commit of AlertPublishers (Provisional)

i

aefoiajwoef

initial draft to publisher crap

awefla

aewlfkjhawiehgsv

Add some alerts

aweklrgjhsakf

fadsuhiawef

Adds some basic publisher framework

pylint

Compoiste publisher

Assemble_alert_publisher_for_output

more code

fix tests

Replace with publishers

WORKING COMMIT

This commit is proven to work on Stage.

TODOs:
- publishers are hard to code, and the class system seems redundant?
- merging needs to be tested
- required outputs will get published if blanket publication is enabled

Fixes a bug where publishers are assigned to required outputs

Tidies up docblocks

Flesh out publisher documentation

* Extends SlackOutput to support custom messages and attachments

* Refactor publishers into new directory

Functional commit

* Continues to fumble with classes and namespaces and stuff

* cyclial python module dependency

* Missing mock_s3

* workign commit fumbling with more namespaces and directories

* pylint

* Publishers working?

* w

* Fixups

* Fix comment

* fix

* Consolidate some functions, rename some Classes to be more compact

* more touchiups

* i

* Update documentation

* Refactor to move DefaultPublisher into the core

* Add test coverage

* Address unused-argument

* Remove some extraneous import module things

* Move core.py publisher code into shared

* fix comment

* Catch, log, and reraise keyerror in composite publisher

* Fix buggy docuemntationt

* Fix docvumentation

* @Rule -> @rule

* fix documentation string

* Reverses the order specific and unspecific output publishers are executed to be more intuitive

* Move import_folders to new module

* Remove deepcopy, delegating it to CompositePublisher

* Clean up docblocks

* Remove extraneous return

* Write a test for chained inheritance... (continued)

633 of 633 new or added lines in 17 files covered. (100.0%)

5036 of 5211 relevant lines covered (96.64%)

0.97 hits per line