Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add config option to disable SMTP auth for basic postfix setup #314

Merged
merged 2 commits into from
Dec 18, 2021

Conversation

unsocial-bleach
Copy link
Contributor

Sometimes it is useful to disable SMTP authentication for sending emails. For example, if someone is using a local postfix server, the default install doesn't require/enable auth.

When this new config option is set to enable_smtp_auth: false, the username_smtp and password_smtp config options are ignored.

@martomi martomi changed the base branch from main to dev December 18, 2021 23:15
Copy link
Owner

@martomi martomi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@martomi martomi merged commit b4342fe into martomi:dev Dec 18, 2021
martomi pushed a commit that referenced this pull request Jul 2, 2022
Sometimes it is useful to disable SMTP authentication for sending emails. For example, if someone is using a local postfix server, the default install doesn't require/enable auth.

When this new config option is set to enable_smtp_auth: false, the username_smtp and password_smtp config options are ignored.
martomi pushed a commit that referenced this pull request Jul 2, 2022
* Tests: Add block found tests (#278)

* Parsers: Fix block parser (#293)

* Bugfix: Check for non MacOS operating systems before running apt-get

* Add Pushcut notifier as an alternative to Pushover (#304)

* Add IFTTT notifying functionality  (#305)

* Config: Toggle notifications for plot increases and decreases (#307)

This commits adds new notification for plot increases (e.g connecting HDD) and makes the notifications for increases/decreases configurable via the config for every integration.

* Add a sandboxed systemd service

* Add a scripts/linux/chiadog.service systemd example that attempts to run chiadog in a more isolated environment. Create a new limited user each run (making much of the filesystem readonly), and set `.chia/mainnet` folders other than `log` to inaccessible.
* Move the offset file - previously, the `debug.log.offset` file was kept in the chiadog directory, but when running in a read only filesystem we can't write there. Create a temporary directory when running, and store the offset file there. This also means we no longer need to delete the offset file on startup.

* Add ~/.chia_keys to list of InaccessiblePaths in systemd

Chia is [migrating their keyfiles](https://github.com/Chia-Network/chia-blockchain/wiki/Passphrase-Protected-Chia-Keys-and-Key-Storage-Migration) to `~/.chia_keys`. Block this folder from access in the systemd service.

* Add config option to disable SMTP auth for basic postfix setup (#314)

Sometimes it is useful to disable SMTP authentication for sending emails. For example, if someone is using a local postfix server, the default install doesn't require/enable auth.

When this new config option is set to enable_smtp_auth: false, the username_smtp and password_smtp config options are ignored.

* Add a transaction amount filter

The filter was added primarily to deal with Chia dust storms which spam
wallets with large amounts of transactions that contain tiny amounts of
XCH. The configuration is done via an optional top level filter section
in config.yaml with an initial transaction_amount to allow for
additional future filter configurations.

* Change config structure to handlers.wallet_added_coin_handler

* Move documentation to config-example.yaml

* Add log statement when a chia message gets filtered out

* Add log statement/s when wallet_added_coin_handler is enabled plus
refactor config checking

* Add failing test

Co-authored-by: Kyle Yee <kanasite@users.noreply.github.com>
Co-authored-by: Connor Knabe <Connor-Knabe@users.noreply.github.com>
Co-authored-by: Alex King <alexking@me.com>
Co-authored-by: Pamela R. Hunt <90120449+unsocial-bleach@users.noreply.github.com>
martomi pushed a commit that referenced this pull request Jul 2, 2022
* Tests: Add block found tests (#278)

* Parsers: Fix block parser (#293)

* Bugfix: Check for non MacOS operating systems before running apt-get

* Add Pushcut notifier as an alternative to Pushover (#304)

* Add IFTTT notifying functionality  (#305)

* Config: Toggle notifications for plot increases and decreases (#307)

This commits adds new notification for plot increases (e.g connecting HDD) and makes the notifications for increases/decreases configurable via the config for every integration.

* Add a sandboxed systemd service

* Add a scripts/linux/chiadog.service systemd example that attempts to run chiadog in a more isolated environment. Create a new limited user each run (making much of the filesystem readonly), and set `.chia/mainnet` folders other than `log` to inaccessible.
* Move the offset file - previously, the `debug.log.offset` file was kept in the chiadog directory, but when running in a read only filesystem we can't write there. Create a temporary directory when running, and store the offset file there. This also means we no longer need to delete the offset file on startup.

* Add ~/.chia_keys to list of InaccessiblePaths in systemd

Chia is [migrating their keyfiles](https://github.com/Chia-Network/chia-blockchain/wiki/Passphrase-Protected-Chia-Keys-and-Key-Storage-Migration) to `~/.chia_keys`. Block this folder from access in the systemd service.

* Add config option to disable SMTP auth for basic postfix setup (#314)

Sometimes it is useful to disable SMTP authentication for sending emails. For example, if someone is using a local postfix server, the default install doesn't require/enable auth.

When this new config option is set to enable_smtp_auth: false, the username_smtp and password_smtp config options are ignored.

* Add a transaction amount filter

The filter was added primarily to deal with Chia dust storms which spam
wallets with large amounts of transactions that contain tiny amounts of
XCH. The configuration is done via an optional top level filter section
in config.yaml with an initial transaction_amount to allow for
additional future filter configurations.

* Change config structure to handlers.wallet_added_coin_handler

* Move documentation to config-example.yaml

* Add log statement when a chia message gets filtered out

* Add log statement/s when wallet_added_coin_handler is enabled plus
refactor config checking

* Add failing test

Co-authored-by: Kyle Yee <kanasite@users.noreply.github.com>
Co-authored-by: Connor Knabe <Connor-Knabe@users.noreply.github.com>
Co-authored-by: Alex King <alexking@me.com>
Co-authored-by: Pamela R. Hunt <90120449+unsocial-bleach@users.noreply.github.com>
martomi pushed a commit that referenced this pull request Jul 2, 2022
Sometimes it is useful to disable SMTP authentication for sending emails. For example, if someone is using a local postfix server, the default install doesn't require/enable auth.

When this new config option is set to enable_smtp_auth: false, the username_smtp and password_smtp config options are ignored.
jinnatar pushed a commit to jinnatar/chiadog that referenced this pull request Feb 5, 2023
…mi#314)

Sometimes it is useful to disable SMTP authentication for sending emails. For example, if someone is using a local postfix server, the default install doesn't require/enable auth.

When this new config option is set to enable_smtp_auth: false, the username_smtp and password_smtp config options are ignored.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants