Skip to content

Conversation

vseanreesermsft
Copy link

No description provided.

rzikm and others added 18 commits June 11, 2025 10:21
…d v4

Updated prepare-signed-artifacts.yml - use microbuild v4

----
#### AI description  (iteration 1)
#### PR Classification
This pull request updates a CI pipeline configuration by upgrading the MicroBuildSigningPlugin version.

#### PR Summary
This pull request modifies the pipeline YAML to use MicroBuildSigningPlugin version 4 instead of version 2 and adds a new parameter for the connected PME service.
- `eng/pipelines/official/jobs/prepare-signed-artifacts.yml`: Updated the task version from `MicroBuildSigningPlugin@2` to `MicroBuildSigningPlugin@4` and added the `ConnectedPMEServiceName` parameter with a specified service ID.
<!-- GitOpsUserAgent=GitOps.Apps.Server.pullrequestcopilot -->
@Copilot Copilot AI review requested due to automatic review settings October 14, 2025 18:44
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds early failure detection for SMTP connections that require SSL/TLS when the server doesn't support EHLO. The change prevents attempting HELO commands when SSL is enabled and EHLO has failed, since HELO cannot advertise STARTTLS capability.

  • Adds validation to fail early when SSL is required but server only supports HELO
  • Improves error handling by throwing a clear exception about STARTTLS not being supported

@jozkee jozkee added the NO-SQUASH The PR should not be squashed label Oct 14, 2025
@PranavSenthilnathan PranavSenthilnathan added the Servicing-approved Approved for servicing release label Oct 14, 2025
@PranavSenthilnathan PranavSenthilnathan merged commit 46d412f into dotnet:release/8.0 Oct 14, 2025
110 of 116 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

NO-SQUASH The PR should not be squashed Servicing-approved Approved for servicing release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants