fix: Email tracking without "use_ssl" (#26718) (#26735)

Everyone uses TLS these days, idk why this config exists.

closes #26717

(cherry picked from commit a65c621)

Co-authored-by: Ankush Menat <ankush@frappe.io>

frappe/email/doctype/email_queue/email_queue.py

@@ -236,6 +236,7 @@ def __init__(
 		self.sent_to_atleast_one_recipient = any(
 			rec.recipient for rec in self.queue_doc.recipients if rec.is_mail_sent()
 		)
+		self.email_account_doc = None
 
 	def fetch_smtp_server(self):
 		self.email_account_doc = self.queue_doc.get_email_account(raise_error=True)
@@ -325,7 +326,11 @@ def get_tracker_str(self, recipient_email) -> str:
 			}
 			tracker_url = get_url(f"{email_read_tracker_url}?{get_signed_params(params)}")
 
-		elif frappe.conf.use_ssl and self.email_account_doc.track_email_status:
+		elif (
+			self.email_account_doc
+			and self.email_account_doc.track_email_status
+			and self.queue_doc.communication
+		):
 			tracker_url = f"{get_url()}/api/method/frappe.core.doctype.communication.email.mark_email_as_seen?name={self.queue_doc.communication}"
 
 		if tracker_url:

frappe/tests/test_email.py

@@ -119,7 +119,6 @@ def test_cc_header(self):
 		self.assertTrue("CC: test1@example.com" in message)
 
 	def test_cc_footer(self):
-		frappe.conf.use_ssl = True
 		# test if sending with cc's makes it into header
 		frappe.sendmail(
 			recipients=["test@example.com"],
@@ -151,10 +150,6 @@ def test_cc_footer(self):
 			in frappe.safe_decode(frappe.flags.sent_mail)
 		)
 
-		# check for email tracker
-		self.assertTrue("mark_email_as_seen" in frappe.safe_decode(frappe.flags.sent_mail))
-		frappe.conf.use_ssl = False
-
 	def test_expose(self):
 		from frappe.utils import set_request
 		from frappe.utils.verified_command import verify_request