It would be nice if at least the code to create
X-Downloaded-From: header
Could get added into 4.2#1 with a complete fix in 4.2#2
Thanks
Robert
This has been tremendously useful for us. We put it in as a "Downloaded-Through" header. Please make this standard! :)
For what it's worth, the code we used is here:
** in 4.2 -- /var/www/html/cerb4/plugins/cerberusweb.core/api/cron.classes.php
on Line 1111
We inserted
fwrite($fp,"\r\nDownloaded-From: $account->nickname");
So, the section it's in now looks like this.
Lines 1110-1115:
if($fp) {
fwrite($fp,"Downloaded-From: $account->nickname\r\n");
fwrite($fp,$headers,strlen($headers));
fwrite($fp,"\r\n\r\n");
fwrite($fp,$body,strlen($body));
@fclose($fp);
}
Of course this should probably be turned into a regular header and handled with the rest of the list, but that's how it's working now.
Robert deserves the credit for this fix; he did 90% of it. Don't want to be a glory hound here or anything.
With the likely inclusion of
http://www.wgmdev.com/jira/browse/CHD-1353 in either 4.2.4 or 4.3.0 a plugin could get us 90% of the way. We just need a header recording the account recording that the email was download from. This could be inserted without a change in interface. If you record a time stamp when download you could allow people to resolve the question of client date/time vers. download date/time giving the use the option of having a plugin that can reset that time and date inside a plug in.
Thanks
Robert
Patch agaist current Cerb5 development.
I am not mantaining a branch to track this patcch.
For Cerb5.
http://github.com/rmiddle/cerb5/tree/jira/chd-1218
For Cerb4. It is in my master I will create a new comment when I break it out.
Thanks
Robert
Updated title to reflect the real change that is needed.
Thanks
Robert
Thanks! I'll get this implemented.
Patched in 5.6.3-dev