mirror of
https://github.com/anonaddy/anonaddy.git
synced 2026-04-25 14:15:53 +03:00
[GH-ISSUE #60] Self Hosted | Email bounce when fowarded/Domain part missing #660
Labels
No labels
bug
pull-request
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/anonaddy#660
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @slurdge on GitHub (Aug 19, 2020).
Original GitHub issue: https://github.com/anonaddy/anonaddy/issues/60
When forwarding the mail, the sender is rewritten but it bounces like so:
My guess is that the domain part is missing and thus not added after the part that should be before the @.
Related to this, the new aliases miss the domain part too (but are active).
Could it come from the fact that
ANONADDY_DOMAINandANONADDY_HOSTNAMEare the same domain ?@slurdge commented on GitHub (Aug 19, 2020):
To add to it, here is what i see when I have users.
I don't understand from the code where the domain part is populated from. Probably it's lost somewhere or there is a step that I missed.
@willbrowningme commented on GitHub (Aug 20, 2020):
Have you populated the
ANONADDY_ALL_DOMAINS.env variable? This is just a list of all the domains you are using and want to allow other users to be able to use too.@willbrowningme commented on GitHub (Aug 20, 2020):
It should be a comma separated list in your .env file like this:
ANONADDY_ALL_DOMAINS=anonaddy.com,anonaddy.me@slurdge commented on GitHub (Aug 20, 2020):
Yes that was it! I just tried and everything works as expected. I actually tried it while you commented, but I couldn't find it in the docs (apart from the .env.example file).
@crazy-max @willbrowningme Maybe we can derive this variable from the ANONADDY_DOMAIN variable if the ALL_DOMAINS one is empty ?
@crazy-max commented on GitHub (Aug 20, 2020):
@slurdge
I think this should be done directly through the app? WDYT @willbrowningme?
@willbrowningme commented on GitHub (Aug 20, 2020):
Yes I think that would be best, the relevant code is line 328 of the User model.
https://github.com/anonaddy/anonaddy/blob/master/app/User.php#L328
I'll push an update shortly.
@willbrowningme commented on GitHub (Aug 20, 2020):
Updated in v0.2.10 - https://github.com/anonaddy/anonaddy/releases/tag/v0.2.10