Skip to content

add an environment variable for the smtp_pass config option #1138

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

Merged
merged 1 commit into from
Jul 27, 2025

Conversation

SigmaSquadron
Copy link
Contributor

Community Contribution License Agreement

By creating this pull request, I grant the project maintainers an unlimited,
perpetual license to use, modify, and redistribute these contributions under any terms they
choose, including both the AGPLv3 and the Fossorial Commercial license terms. I
represent that I have the right to grant this license for all contributed content.

Description

The password for secure authentication may be sensitive, so it is best to not leave it lying around in a config file. This commit introduces the EMAIL_SMTP_PASS environment variable, which can be set to configure the SMTP password without writing it to the configuration file.

How to test?

Start a Pangolin server as normal, but use EMAIL_SMTP_PASS instead of the config file to set up the mail server.

The password for secure authentication may be sensitive, so it is best
to not leave it lying around in a config file. This commit introduces
the EMAIL_SMTP_PASS environment variable, which can be set to configure
the SMTP password without writing it to the configuration file.

Signed-off-by: Fernando Rodrigues <alpha@sigmasquadron.net>
SigmaSquadron added a commit to SigmaSquadron/fosrl-docs that referenced this pull request Jul 27, 2025
Documentation part of fosrl/pangolin#1138

Signed-off-by: Fernando Rodrigues <alpha@sigmasquadron.net>
@oschwartz10612 oschwartz10612 changed the base branch from main to dev July 27, 2025 17:08
@oschwartz10612 oschwartz10612 changed the base branch from dev to main July 27, 2025 17:08
@oschwartz10612 oschwartz10612 merged commit 8e45c34 into fosrl:main Jul 27, 2025
@oschwartz10612
Copy link
Member

Thanks!

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