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

Bring back exception serialization support #23732

Closed
7 tasks done
ViktorHofer opened this issue Oct 4, 2017 · 6 comments
Closed
7 tasks done

Bring back exception serialization support #23732

ViktorHofer opened this issue Oct 4, 2017 · 6 comments
Assignees
Labels
area-System.Runtime enhancement Product code improvement that does NOT require public API changes/additions
Milestone

Comments

@ViktorHofer
Copy link
Member

We see strong customer demand for being able to serialize Exceptions with BinaryFormatter or DCS.

The following actions are required:

cc @jkotas @stephentoub @danmosemsft

@ViktorHofer ViktorHofer self-assigned this Oct 4, 2017
@4creators
Copy link
Contributor

4creators commented Oct 4, 2017

With which version will this change be shipped?

@danmoseley
Copy link
Member

@4creators definitely in 2.1. We are exploring the possibility of servicing 2.0 for it. Generally we don't service for "features" but this is necessary for some customers to port successfully. Do you need it?

@4creators
Copy link
Contributor

Yeah, unfortunately I need it for all custom exceptions I use in every library I work on 😒

@danmoseley
Copy link
Member

@4creators custom exceptions in general should be serializable already, as in 2.0 we made Exception do all necessary work to serialize. They just need to call the base. The problem is other exceptions we ship.

@4creators
Copy link
Contributor

4creators commented Oct 4, 2017

@danmosemsft My implementation pattern for throwing exceptions is to derive from every framework exception I want to throw my own exception i.e. CustomInvalidOperationException etc. Therefore Exception is not enough and I can not derive from Exception either as this would be braking change for error handling code where I do it seperatly for my CustomInvalidOperationException and framework InvalidOperationException. And I have a lot's of them and a lot of error handling paths.

@danmoseley
Copy link
Member

Got it.

@msftgits msftgits transferred this issue from dotnet/corefx Jan 31, 2020
@msftgits msftgits added this to the 2.1.0 milestone Jan 31, 2020
@ghost ghost locked as resolved and limited conversation to collaborators Dec 20, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-System.Runtime enhancement Product code improvement that does NOT require public API changes/additions
Projects
None yet
Development

No branches or pull requests

4 participants