Skip to content

issue with existing alias which doesn't exist#3259

Closed
bibiak1 wants to merge 1 commit intodocker-mailserver:masterfrom
bibiak1:master
Closed

issue with existing alias which doesn't exist#3259
bibiak1 wants to merge 1 commit intodocker-mailserver:masterfrom
bibiak1:master

Conversation

@bibiak1
Copy link
Copy Markdown

@bibiak1 bibiak1 commented Apr 14, 2023

Description

Issue with adding new recipient to existing alias
If new recipient exists as alias in db then setup is throwing error.

Fixes part of script checks only the relevant alias and compares recipients for it.

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Improvement (non-breaking change that does improve existing functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation (README.md or the documentation under docs/)
  • If necessary I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

@georglauterbach
Copy link
Copy Markdown
Member

georglauterbach commented Apr 14, 2023

Thank you for providing a PR for an issue you encountered. Usually, we require users to open an issue first to discuss the matter, because:

If new recipient exists as alias in db then setup is throwing error.

That behavior seems reasonable to me. Why would you change that? Can you describe in more detail what you mean here?


It would be very nice of you could open a new bug report issue and describe your problem in more detail there. Thank you :)

Copy link
Copy Markdown
Member

@georglauterbach georglauterbach left a comment

Choose a reason for hiding this comment

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

Blocking this until discussion about actual issue is resolved.

Moreover,

  • If necessary I have added tests that prove my fix is effective or that my feature works

Have you? I don't see them.

@bibiak1
Copy link
Copy Markdown
Author

bibiak1 commented Apr 14, 2023

#3262

@bibiak1
Copy link
Copy Markdown
Author

bibiak1 commented Apr 18, 2023

It's now an issue. It is a email forwarding feature. How to forward email if you blocked alias for email account?

@georglauterbach
Copy link
Copy Markdown
Member

georglauterbach commented Apr 18, 2023

How to forward email if you blocked alias for email account?

We did not block alias for all email accounts, we blocked aliases that are equal to accounts.

It's now an issue. It is a email forwarding feature.

Not quite; you're not actually forwarding / relaying an email, because this would imply that either DMS is the new sender (forwarding) or DMS is not the addressee but an intermediate MTA (relaying). You somehow want Gmail to act on DMS. @polarathene has proposed a solution in #3262 (comment) for the issue you raised.

@bibiak1
Copy link
Copy Markdown
Author

bibiak1 commented Apr 18, 2023

Not quite; you're not actually forwarding / relaying an email, because this would imply that either DMS is the new sender (forwarding) or DMS is not the addressee but an intermediate MTA (relaying). You somehow want Gmail to act on DMS. @polarathene has proposed a solution in #3262 (comment) for the issue you raised.

It was not my concern that there's an alias for account. I raised other concern

@polarathene
Copy link
Copy Markdown
Member

How to forward email if you blocked alias for email account?

I think what you would like is relay_domains support. This allows a separate transport to relay mail received.

If it is not acceptable to store mail on DMS and retrieve it like suggested, this should let mail for [email protected] to arrive and be relayed to [email protected] automatically instead.

You can restrict the outgoing addresses that get relayed this way via relay_recipient_maps (you'd want this, to avoid spammers abusing it). See example config documented here. It is not supported by DMS, feature would need to be added or manually manage the config yourself.

You'll probably also need smtp_generic_maps which is what maps @example.com to @gmail.com for outgoing mail. If you want to keep the original FROM / Sender address, then you'll probably need PostSRSd I think since you'd otherwise need SPF approval from everyone sending mail to you for you to relay on their behalf.

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.

3 participants