Fix single email

[CTY-git]

PR Checklist

• The commit message follows our guidelines: Code of conduct
• Tests for the changes have been added (for bug fixes / features)
• Docs have been added / updated (for bug fixes / features)
• Does this PR introduce a breaking change?
• Include PR in release notes?

PR Type

• Bugfix
• Feature
• Refactoring
• Build /CI
• Documentation
• Others

What is the current behavior?

Issue Number: N/A

What is the new behavior?

Other information

[patched-admin]

File Changed: patchwork/steps/SendEmail/SendEmail.py

Rule 1: Do not ignore potential bugs in the code

Details: The removal of parse_to_list function and modification of recipient email handling introduces a potential bug. The code previously supported multiple email recipients through parsing space or comma-separated lists, but now assumes a single string input. This could break existing functionality where multiple recipients were being specified.

Affected Code Snippet:

- self.recipient_email = parse_to_list(inputs["recipient_email"], [" ", ","])
+ self.recipient_email = inputs["recipient_email"]

- msg["To"] = ", ".join(self.recipient_email)
+ msg["To"] = self.recipient_email

Start Line: 18
End Line: 31

---

Rule 2: Do not overlook possible security vulnerabilities

Details: The code modification has removed input validation that was previously provided by the parse_to_list function. This could potentially allow malformed or malicious email addresses to be processed without proper sanitization, which might lead to email header injection attacks.

Affected Code Snippet:

- self.recipient_email = parse_to_list(inputs["recipient_email"], [" ", ","])
+ self.recipient_email = inputs["recipient_email"]

Start Line: 18
End Line: 18