Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[rel/3.8] Fix serialization of exceptions by BinaryFormatter in .NET Framework #5055

Merged

Conversation

youssef-backport-bot
Copy link

Backport of #5054 to rel/3.8

/cc @Evangelink @nohwnd

Evangelink
Evangelink previously approved these changes Feb 18, 2025
@Evangelink Evangelink enabled auto-merge (squash) February 18, 2025 09:26
@nohwnd
Copy link
Member

nohwnd commented Feb 18, 2025

Failed on output not mixed tests, looks like flakiness.

@Youssef1313
Copy link
Member

test\UnitTests\TestFramework.UnitTests\Exceptions\BinaryFormatterExceptionSerializationTests.cs(22,48): error CS0618: (NETCORE_ENGINEERING_TELEMETRY=Build) 'InternalTestFailureException' is obsolete: 'We will remove or hide this type starting with v4. If you are using this type, reach out to our team on ***.'

@nohwnd
Copy link
Member

nohwnd commented Feb 18, 2025

I was looking specifically for that in local run of unit tests and saw just the integration tests fail. Same as here in the failed tests:
image

dammit such complicated reporting (not your fault of course).

@nohwnd
Copy link
Member

nohwnd commented Feb 18, 2025

fixing on the original, where we are also discussing the approach.

auto-merge was automatically disabled February 18, 2025 15:02

Head branch was pushed to by a user without write access

@Evangelink Evangelink enabled auto-merge (squash) February 18, 2025 15:44
@Evangelink Evangelink merged commit ac22a6d into microsoft:rel/3.8 Feb 18, 2025
8 checks passed
@Youssef1313 Youssef1313 deleted the backport/5054/to/rel/3.8 branch February 18, 2025 16:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants