cloudmarker / cloudmarker

94%

master: 84%

LAST BUILD ON BRANCH emaildocs

branch: emaildocs

CHANGE BRANCH

x

Reset

• emaildocs
• 0.1.0
• alertdocs
• apidocindent
• audit-version
• auditemails
• azcloud
• azstorage
• azurecloud
• azurecloud-exception
• azvm_record_type
• azwebapp
• badges
• baseconfig
• bracket
• buckets
• build-matrix
• changelog
• clioptions
• clouderror
• common-plugins-config
• consistency
• copy-ext
• copyext
• cov_clean
• dependabot/pip/cryptography-3.3.2
• dependabot/pip/httplib2-0.18.0
• dependabot/pip/httplib2-0.19.0
• dependabot/pip/pyyaml-5.4
• dependabot/pip/rsa-4.7
• dependabot/pip/urllib3-1.26.5
• dist
• docfixes
• doctest-fix
• email_debug_param_from_config
• emailbody
• empty-port
• esstore
• firewalleventtest
• firewallnorm
• firewallruleevent
• fix-esstore-done
• fix-unused-input-queue
• fix_cloud_worker
• fixbuild
• fixcoverage
• fixindent
• freeze
• gcpcloudlogs
• gcpcloudtest
• iam_policy
• ignore-virtual-machines
• ioworkers
• key_file_name_fix
• lint
• load_yaml_safely
• log
• logfixes
• logger-defaults
• logging-levels
• master
• maxrecs
• mock-records
• mockplugins
• mongodbdefaults
• mongodbfix
• mongodbtests
• multiprocessing
• pluginframework
• pluginloader
• readable-version
• readme
• remove-email-config
• removefirewallevent
• rename_logger
• renameparams
• requirements
• rest
• reuse-mock
• rm-venv
• slackalert
• smtpssl
• sphinx
• sphinx-rst-update
• splunkfixes
• splunkhecstore_docs
• splunkhectokentest
• splunkstore
• sub_id_rename
• tests
• travis
• userconfig
• userdoc
• userecordterm
• verdev
• webapp
• worker-com-fields
• worker-error-handling
• workerlogs

Build # 622

Build Type

Pull #98

travis-ci-com

Committed by web-flow

Commit Message

Make EmailAlert docs and code consistent

The EmailAlert docstrings have been updated to make it look good from a
user's perspective. In some places, the docstrings have been fixed to
describe the functionality in the code correctly. In some other cases,
the code has been updated to be consistent with the updated docstrings.

The following changes have been made to the plugin parameters:

  - Rename `sender` to `from_addr`.
  - Rename `to` to `to_addrs`.
  - Remove `body`.
  - Add defaults for `subject`, `host`, and `port`.

The names `from_addr` and `to_addrs` have been picked from the Python
`smtplib` documentation. They have done the work of choosing good names,
so we just reuse it. The `to_addrs` name has an additional advantage
that it hints at the value being a list of addresses.

The `body` parameter has been removed because this is not a generic
emailer class. This is an alert plugin which will decide its own email
content based on the events it receives in its `write()` method.
There is never a situation where the email content can be specified via
configuration.

The defaults for `host` and `port` have also been picked from the Python
`smtplib` documentation. They implement sensible default behaviour for
these defaults, so we reuse that behaviour by choosing the same
defaults.

Pull Request Pull Request #98: Make EmailAlert docs and code consistent

Run Details

323 of 340 branches covered (95.0%)

Branch coverage included in aggregate %.

8 of 8 new or added lines in 1 file covered. (100.0%)

1566 of 1659 relevant lines covered (94.39%)

2.83 hits per line