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

Reorder smtpd_sender_restrictions #882

Open
wants to merge 1 commit into
base: stretch-unstable
from

Conversation

@yalh76
Copy link
Contributor

yalh76 commented Feb 19, 2020

To let send without login when coming from localhost

The problem

When trying to send mail from apps on Yunohost, for now the app need to first authenticate, which means: have to create a dedicated user for the app

Solution

Reorder smtpd_sender_restrictions to allow to send mail without login from mynetworks

PR Status

Ready

How to test

For example, install mobilizon_ynh and try to register a user. Without this patch, you will find in /var/log/mail.info something like NOQUEUE: reject: RCPT from localhost[127.0.0.1]: 553 5.7.1 <xxx@xxxx.xxx>: Sender address rejected: not logged in;

With the patch, all goes well, mail sent and received

Validation

  • Principle agreement 0/2 :
  • Quick review 0/1 :
  • Simple test 0/1 :
  • Deep review 0/1 :
To let send without login when coming from localhost
@rds13

This comment has been minimized.

Copy link
Contributor

rds13 commented Feb 20, 2020

👍

@kay0u
kay0u approved these changes Feb 23, 2020
Copy link
Member

kay0u left a comment

LGTM

@yalh76

This comment has been minimized.

Copy link
Contributor Author

yalh76 commented Mar 17, 2020

Maybe to be tested if it doesn't allow a user using rainloop or roundcube to send mails without authentication...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.