To recipients keep getting added on subsequent emails

[Ashish Khanal]

I'm using FluentEmail to send emails, everything is working well except there's this issue where the 'To' recipients keep getting added for every subsequent email.

For eg: If it's sending emails to [e-mail]:

1. The first time the SendEmailAsync method is called, it sends email to: [e-mail]

   

2. The second time the SendEmailAsync method is called, it sends email to: [e-mail];[e-mail]

   

3. The third time the SendEmailAsync method is called, it sends email to:
   [e-mail];[e-mail];[e-mail]

   

and so on. It gets really long after some time.

My code looks like this:

ConfigureServices method:

// Set email service using FluentEmail
services.AddFluentEmail("[e-mail]")
        .AddRazorRenderer(@$"{Directory.GetCurrentDirectory()}/Views/")
        .AddSmtpSender("smtp.somecompanyname.com", 25)
        .AddSmtpSender(new System.Net.Mail.SmtpClient() { });

Now the email service looks like this:

public class FluentEmailService : IFluentEmailService
{
    private readonly IFluentEmail _fluentEmail;
    private readonly ILogger<FluentEmailService> _logger;
    public FluentEmailService(ILogger<FluentEmailService> logger, IFluentEmail fluentEmail)
    {
        _logger = logger;
        _fluentEmail = fluentEmail;
    }

    public async Task<SendResponse> SendEmailAsync<TModel>(string subject, string razorTemplatePath, TModel model, string semicolonSeparatedEmailRecipients)
    {
		var sendResponse = await _fluentEmail
						.To(semicolonSeparatedEmailRecipients)
						.Subject(subject)
						.UsingTemplateFromFile(razorTemplatePath, model)
						.SendAsync();
		return sendResponse;
    }
}

[McPhale]

I'm not sure, but I think this could be resolved by using IFluentEmailFactory instead of IFluentEmail.

[Ashish Khanal]

I'm not sure, but I think this could be resolved by using IFluentEmailFactory instead of IFluentEmail.

Thank you for the answer @McPhale. That exactly solved my issue.

Now the email service looks like this:

public class FluentEmailService : IFluentEmailService
{
    private readonly IFluentEmailFactory _fluentEmailFactory;
    private readonly ILogger<FluentEmailService> _logger;
    public FluentEmailService(ILogger<FluentEmailService> logger, IFluentEmailFactory fluentEmailFactory)
    {
        _logger = logger;
        _fluentEmailFactory = fluentEmailFactory;
    }

    public async Task<SendResponse> SendEmailAsync<TModel>(string subject, string razorTemplatePath, TModel model, string semicolonSeparatedEmailRecipients)
    {
		var sendResponse = await _fluentEmailFactory
                                                .Create()
						.To(semicolonSeparatedEmailRecipients)
						.Subject(subject)
						.UsingTemplateFromFile(razorTemplatePath, model)
						.SendAsync();
		return sendResponse;
    }
}