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

additional_message_headers plugin in sent read receipts #4302

Closed
rcubetrac opened this issue Aug 15, 2013 · 6 comments
Closed

additional_message_headers plugin in sent read receipts #4302

rcubetrac opened this issue Aug 15, 2013 · 6 comments

Comments

@rcubetrac
Copy link

Reported by jasb on 15 Aug 2013 11:03 UTC as Trac ticket #1489284

Hi,

The "additional_message_headers" works OK, but in sent read receipts sent from the webmail, the headers are not included.

Shouldn't they be included in read receipts as in regular messages?

Migrated-From: http://trac.roundcube.net/ticket/1489284

@rcubetrac
Copy link
Author

Comment by @alecpl on 15 Aug 2013 11:18 UTC

I suppose they should.

@rcubetrac
Copy link
Author

Milestone changed by @alecpl on 15 Aug 2013 11:18 UTC

later => 0.9.3

@rcubetrac
Copy link
Author

Comment by @thomascube on 16 Aug 2013 08:44 UTC

The message_outgoing_headers hook call should probably be moved to rcube::deliver_message() although the current implementation also modifies headers when saving drafts.

@rcubetrac
Copy link
Author

Comment by @alecpl on 24 Aug 2013 17:14 UTC

We have already message_before_send hook in deliver_message(). So, I propose to get rid of message_outgoing_headers hook and use message_before_send instead.

@rcubetrac
Copy link
Author

Comment by @alecpl on 25 Aug 2013 07:16 UTC

Fixed in 6efadfc.

@rcubetrac
Copy link
Author

Status changed by @alecpl on 25 Aug 2013 07:16 UTC

new => closed

@rcubetrac rcubetrac added this to the 0.9.4 milestone Mar 20, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant