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

Feature request - enable smtp auth #22

Closed
kaislaoja opened this issue Aug 25, 2022 · 6 comments
Closed

Feature request - enable smtp auth #22

kaislaoja opened this issue Aug 25, 2022 · 6 comments
Assignees
Labels
enhancement New feature or request

Comments

@kaislaoja
Copy link

I would like to see a way to use some kind of authentication when you send emails to the mailrise. At least some type of local username/password list in the configuration file or maybe even a LDAP(S) integration if it can be implemented easily.

I know that this is meant to be used in a private network where there shouldn't be unauthorised access, but I still would want to use authentication.

@YoRyan YoRyan added the enhancement New feature or request label Aug 25, 2022
@YoRyan YoRyan self-assigned this Aug 25, 2022
@YoRyan
Copy link
Owner

YoRyan commented Aug 25, 2022

Basic authentication, at least, should be easy.

@ksurl
Copy link

ksurl commented Aug 31, 2022

it would help with applications that don't have a no-auth smtp config option as well.

@YoRyan YoRyan closed this as completed in e789e02 Sep 30, 2022
@YoRyan
Copy link
Owner

YoRyan commented Sep 30, 2022

it would help with applications that don't have a no-auth smtp config option as well.

Yes, that's a good point. Basic authentication is now available in the latest tip.

@ksurl
Copy link

ksurl commented Sep 30, 2022

works great. I had to dig into the commit to find the syntax in the test authenticator mockup.

smtp:
  auth:
    basic:
      username: password

@ksurl
Copy link

ksurl commented Sep 30, 2022

It does not pick up config changes while running anymore? I have to restart for it to update. no output in the log either from docker logs mailrise

@YoRyan
Copy link
Owner

YoRyan commented Sep 30, 2022

Mailrise never had hot reloads; you'd always have to restart the container to pick up any config changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants