Add a "From" email address to email Report task

[fahadash (fahadash)]

Currently there is no way to specify in the configuration what do you want email's from address to be. It assumes that the SMTP's username is the from address. But for SMTP service connections where username is not an actual email adddress (does not have @ or domain), this tasks fail at runtime.

This PR will Fix #42 by giving user an optional From email configuration field. If skipped, it will defer back to using SMTP's username. This has minimal impact.

[Deleted user (ghost)]

All CLA requirements met.

[Marcin Mikołajczak (TripleEmcoder)]

Looks good to me. The field was added the same way as other exiting fields.

[Marcin Mikołajczak (TripleEmcoder)]

Well, I tried, but it seems we need someone with true approving powers :)

[fahadash (fahadash)]

Marcin Mikołajczak (@TripleEmcoder) I appreciate that. Until Microsoft decides to wake up and do something about it, I found a workaround. A powershell script that makes use of System.Net.Smtp to connect to your SMTP server and sends that email.

[Marcin Mikołajczak (TripleEmcoder)]

Where do you get the content from? Does the plugin write it somewhere accessible? Could you share the PowerShell script for others to use?

Hopefully someone will get a few notifications now with our activity and find the time to merge the changes.

[fahadash (fahadash)]

Where do you get the content from? Does the plugin write it somewhere accessible? Could you share the PowerShell script for others to use?

Hopefully someone will get a few notifications now with our activity and find the time to merge the changes.

Marcin Mikołajczak (@TripleEmcoder) Sorry for delayed response here is the script that I wrote

$EmailFrom = “no-reply@yourcompany.com”
$EmailTo = “recipients@yourcompany.com”
$Subject = “App Deployment Started”
$Body = “We are taking down the app to deploy the build <a href=`"https://dev.azure.com/Company/projectl/_traceability/runview/changes?currentRunId=$(Build.BuildId)`">$(Build.BuildNumber)</a>, Release $(Release.ReleaseId). ”
$SMTPServer = “smtp.server.com”
$SMTPClient = New-Object Net.Mail.SmtpClient($SmtpServer, 587)
$SMTPClient.EnableSsl = $true
$SMTPClient.Credentials = New-Object System.Net.NetworkCredential("username", "password");
$mail = New-Object System.Net.Mail.Mailmessage $EmailFrom, $EmailTo, $Subject, $Body
$mail.IsBodyHtml = $true

$SMTPClient.Send($mail)

[ksista-ahc (ksista-ahc)]

Any idea when this PR can be merged. There is a definite need to fix this extension fahadash (@fahadash) Microsoft Open Source (@microsoftopensource)

[xiuzhen1103]

Any update on this pull request approval?

[ksista-ahc (ksista-ahc)]

Marcin Mikołajczak (@TripleEmcoder) ddeglin rBuilder-cafeto (@devops-cafeto) Could anyone of you approve the PR and merge it.

[fahadash (fahadash)]

Sai Krishna V (@saikrishnav) Could you merge this PR?

[ksista-ahc (ksista-ahc)]

Sai Krishna V (@saikrishnav) Could you please approve this PR and unblock us... This plugin is a definite need for us

[fahadash (fahadash)]

LeahSchwartz Could you help us out? We need these changes merged ASAP to unblock us.

[Peadge (Peadge)]

LeahSchwartz or anyone else PLEASE can you approve the changes so this can be merged in, it's the ONLY extension that can email test results out from the Release pipeline.

[fahadash (fahadash)]

I have sent an email to opensource@microsoft.com asking for help. Hoping for the best.

[Naweap (Naweap)]

Any news on that PR ?

[fahadash (fahadash)]

I never received a response from Microsoft OpenSource to my email. I am going to try tweeting to their twitter handle, you guys try it too.

[Naweap (Naweap)]

I never received a response from Microsoft OpenSource to my email. I am going to try tweeting to their twitter handle, you guys try it too.

Alright, thanks for your answer, it will try it too. However, I don't want to be pessimistic, but it has been almost one year since they didn't release any new version of this extension. By the time this PR is merged and we see this improvement in prod, I think it's gonna take time.

[Daniel Hoeger (dan-hoeger)]

Is there any update on this? we require the ability to specify the "From" address when using an AWS SMTP service.

[Alexander Sultan-Shakh (asultan80)]

fahadash (@fahadash) Can you remove the review request from ddeglin ?

[Alexander Sultan-Shakh (asultan80)]

Sai Krishna V (@saikrishnav) Please, take a look and help to resolve this one.
Thank you!

[fahadash (fahadash)]

I think we may have to call Microsoft PSS to get their attention into this.

[fahadash (fahadash)]

ddeglin This PR is waiting on you again, could you look into this please?

[fahadash (fahadash)]

Sai Krishna V (@saikrishnav) Could you merge this PR please?

[Zaf (ZaafirDar)]

fahadash (@fahadash) Can we please get this PR merged, still running into this problem.

[fahadash (fahadash)]

Looks like we have lost ddeglin , I am going to convert this to draft, that is probably going to remove the existing approvals.

[fahadash (fahadash)]

It turns out that we have to wait for a Build to occur before this PR can be merged.

[Zaf (ZaafirDar)]

fahadash (@fahadash) Any Idea how is build triggered? Should we expect any response on this?

[fahadash (fahadash)]

Sai Krishna V (@saikrishnav) Could you get the build 10.x check taken care of?

[Zaf (ZaafirDar)]

Sai Krishna V (@saikrishnav) can you help us with build (10.x) Check?

[Andrew Aladjev (andrew-aladev)]

Why this obvious pull request is still opened from 2024 year? Is this repository dead?