derpunker Posted August 1 Share Posted August 1 (edited) Dear IPS support, there is a bug in the bulkmail logic when using SendGrid as external provider. The unsubscribe link in the send email is not valid. The reason for that is the "ClickTracking" logic. What's happening: applications/core/extensions/core/Queue/Bulkmail.php => function "run" calls "mergeAndSend" in SendGrid implementation.  The mergeAndSend calls "compileContent" with member-parameter FALSE  "compileContent" creates the mail content, including the unsubscribe link.  Later in the code/function the "ClickTracking" is added. IMPORTANT HINT: The place holders for email and unsubscribe key are not yet substituded in href.  The "ClickTracking" logic adds an hash value based on the email and unsubscribe key (not yet set).  Back in "mergeAndSend" the placeholders are substituded in the content.  The substituion is to late for the hash calculation in the ClickTracking function during content creation.  As consequence, the unsubscribe link in the email is invalid because of the invalid hash value! Currently, we have multiple support requests because of this issue. This happens only when using SendGrid, the "normal" Email logic is working correctly. The bug invesitgation is done from my end, now it's your turn to fix this bug as soon as possible. 🙂  Hint: There is no need for access to my communitiy suite. The isse can be reproduced in every dev-environment, once SendGrid is activated and you try to send a bulk.  Edited August 1 by derpunker Link to comment Share on other sites More sharing options...
Marc Posted August 2 Share Posted August 2 Thank you for bringing this issue to our attention! I can confirm this should be further reviewed and I have logged an internal bug report for our development team to investigate and address as necessary, in a future maintenance release. Â Link to comment Share on other sites More sharing options...
derpunker Posted August 10 Author Share Posted August 10 @Marc I guess the fix is not included in IPS 4.7.18, right? Link to comment Share on other sites More sharing options...
Randy Calvert Posted August 10 Share Posted August 10 It is not noted in the release notes, so I would venture a guess to say no. And given that releases take several weeks to prep/lock, and that this issue was only confirmed a week ago… I would highly doubt it. Link to comment Share on other sites More sharing options...
Marc Posted August 12 Share Posted August 12 On 8/10/2024 at 5:22 PM, derpunker said: @Marc I guess the fix is not included in IPS 4.7.18, right? I can confirm its not in that version. Link to comment Share on other sites More sharing options...
derpunker Posted August 12 Author Share Posted August 12 (edited) Thanks, I had some hope. 😉 Do you have an estimate in which release the fix will be included? Edited August 12 by derpunker Link to comment Share on other sites More sharing options...
Marc Posted August 13 Share Posted August 13 We are not able to say what versions things will be released in. It will depend on the nature of the fix Link to comment Share on other sites More sharing options...
Recommended Posts