Possible bug in send_email_to_reset_password recipients list

[Alma252]

Hi 👋
I was reading through the tasks.py code and noticed something that might be a bug.

In send_email_to_reset_password, the recipients list is created like this:

recipients = []
recipients.append(user_email)
context["complete_url"] = ...
recipients.append(context["complete_url"])   # <- possible issue?

So the reset URL is also being added to the recipients list, which doesn’t seem correct,
because recipients should only contain email addresses.

Expected behavior:
recipients = [user_email]

But right now, it looks like the reset link is mistakenly added as another recipient.

Could you please check if this is intentional or a bug? 🙏

[RakeshUppunuthula]

@Alma252 Hi I want to work On this issue, but i cant find where the method is located

[Alma252]

Hi :)
Sorry for the confusion, I checked again and the issue is not in send_email_to_reset_password.

The actual issue is in this method:

Path: common/tasks.py
Function: resend_activation_link_to_user

Here is the problematic part:
recipients = [context["complete_url"]]
recipients.append(user_email)

It seems that complete_url (which is a URL, not an email address) is being added to the recipients list.
Since EmailMessage expects a list of email addresses, I believe this is unintended behavior.
The recipients list should only contain email addresses, so it should probably be:

recipients = [user_email]

and the activation link should only be used inside the email content (context), not as an email recipient.
Could you please confirm whether this is intentional or a bug? 🙏
Thanks for your time and for maintaining this project!