Skip to content

Revert "Add SmtpClient.SslOptions for per-client TLS configuration" - #135370

Merged
jkotas merged 1 commit into
mainfrom
revert-134919-rzikm/smtpclient-ssl-options
Oct 7, 2026
Merged

jkotas merged 1 commit into
mainfrom
revert-134919-rzikm/smtpclient-ssl-options

Conversation

@jkotas

@jkotas jkotas commented Oct 7, 2026

Copy link
Copy Markdown
Member

Reverts #134919

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 3 pipeline(s).
13 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

@dotnet-policy-service

Copy link
Copy Markdown
Contributor

Tagging subscribers to this area: @karelz, @dotnet/ncl
See info in area-owners.md if you want to be subscribed.

@jkotas

jkotas commented Oct 7, 2026

Copy link
Copy Markdown
Member Author

/ba-g fixing CI

@jkotas
jkotas merged commit 28013d4 into main Oct 7, 2026
21 of 27 checks passed
@jkotas
jkotas deleted the revert-134919-rzikm/smtpclient-ssl-options branch October 7, 2026 22:08
@dotnet-milestone-bot dotnet-milestone-bot Bot added this to the 12.0-preview1 milestone Oct 8, 2026
rzikm added a commit that referenced this pull request Oct 9, 2026
Reintroduce feature from #134919
that was reverted by #135370 due
to breaking CI too much.

Test failures after merging the original PR were caused by concurrent
changes from #133174 (that landed
in main without us rerunning CI on the original PR). Fixed by disabling
TLS resumption to force certificate validation callbacks to run when
tests depend on them.

Closes #135354

---------

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants