Skip to content
This repository was archived by the owner on Oct 20, 2023. It is now read-only.
This repository was archived by the owner on Oct 20, 2023. It is now read-only.

Config: Notifications:{ enabled: is ignored. Always treated as true. #276

Description

@Heraisu

Description

Setting notifications>enabled to false and restarting the server does not prevent emails from being sent.

Steps to Reproduce

  1. Start with a working config.json from an example.
  2. Set up notification information with a notification server.
  3. Leave valid information, but set "enabled" value to false, as so:

"notifications": {
"enabled": false,
"email": {
"host": "smtp.gmail.com",
"port": 587,
"user": "TheGmailBeingUsed@gmail.com",
"password": "gmailpassword",
"fromAddress": "whatever",
"fromName": "Test Pool"

Expected behavior

No emails are sent when blocks are discovered, etc.

Actual behavior

Emails are still delivered just the same as when enabled:true

Reproduces how often

100%

Version

v21 release and also the head dev build as of 3/18/2018.

Metadata

Metadata

Assignees

No one assigned

    Labels

    Projects

    No projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions