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.

Sign up for GitHub

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

Jump to bottom

Don't set unknown sender as recipient when replying to a mail in Sent folder #9352

Open
phillipp opened this issue Feb 13, 2024 · 1 comment
Open

Don't set unknown sender as recipient when replying to a mail in Sent folder #9352

phillipp opened this issue Feb 13, 2024 · 1 comment
Labels
C: Mail composing enhancement
Milestone
later

Comments

@phillipp
Copy link

Summary

When replying to an email from the "Sent" folder after the sender's identity used for that email has been deleted, the recipient of the reply is incorrectly set to the deleted identity's address instead of the original recipient's address.

Steps to Reproduce:

  1. Send an email to foo@bar.com using a non-default identity (e.g., info@example.com), while the main identity is bob@example.com.
  2. Delete the non-default identity used in step 1 (info@example.com).
  3. Navigate to the "Sent" folder, open the email sent in step 1, and click "Reply".

Expected Behavior: The recipient of the new email in the compose view should be the original recipient (foo@bar.com).
Actual Behavior: The recipient is incorrectly set to the deleted identity's address (info@example.com).

This issue may mirror behavior seen in other email clients, such as Outlook, but it represents an inconsistency with the expected behavior of the email client, where the recipient of a reply should always be the original recipient of the sent email.
This behavior was observed in version 1.6.6.
@alecpl
Copy link
Member

alecpl commented Mar 10, 2024 •
edited
Loading

Confirmed. I don't think we're taking the message folder into consideration when setting the reply recipients. We compare sender with existing identities to tell "it was sent by me". So, in this case "it wasn't sent by me" is concluded.

Maybe in such cases we should take the folder into consideration and act as you describe. Or put both sender and recipient into the recipient field.

EDIT: Thunderbird's behavior is the same as Roundcube's.

@alecpl alecpl added this to the later milestone Mar 10, 2024
@alecpl alecpl added enhancement and removed bug labels Mar 10, 2024
@alecpl alecpl changed the title Replying to sent mail sets sender as recipient when sender identity has been deleted Don't set sender as recipient when replying to a mail in Sent folder Mar 10, 2024
@alecpl alecpl changed the title Don't set sender as recipient when replying to a mail in Sent folder Don't set unknown sender as recipient when replying to a mail in Sent folder Mar 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
C: Mail composing enhancement
Projects
None yet
Milestone
later
Development

No branches or pull requests
2 participants
@alecpl @phillipp