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

Obsolete BinaryFormatter infrastructure #7105

Closed
kant2002 opened this issue Apr 29, 2022 · 3 comments
Closed

Obsolete BinaryFormatter infrastructure #7105

kant2002 opened this issue Apr 29, 2022 · 3 comments
Labels
api-suggestion (1) Early API idea and discussion, it is NOT ready for implementation

Comments

@kant2002
Copy link
Contributor

Is your feature request related to a problem? Please describe

Given that https://docs.microsoft.com/en-us/dotnet/standard/serialization/binaryformatter-security-guide and overall plan to obsolete BinaryFormatter, is there time to do the same and mark constructors with (SerializationInfo info, StreamingContext context). Also all implementations for serialization can be put behind AppContext switch, so if somebody want risky behavior he should opt-in into it.

Example how it is handled with DataTable class https://docs.microsoft.com/en-us/dotnet/fundamentals/syslib-diagnostics/syslib0038

Describe the solution you'd like and alternatives you've considered

Other alternative is provide more blog posts, and guidance how to move out of this technology, and later on again obsolete it and eventually drop support 😄

Will this feature affect UI controls?

Potentially VS designer affected, maybe 3rd party-controls libraries.
They should be informed about change upfront, even if writing was on the wall for a long time.

@kant2002 kant2002 added api-suggestion (1) Early API idea and discussion, it is NOT ready for implementation untriaged The team needs to look at this issue in the next triage labels Apr 29, 2022
@dreddy-work
Copy link
Member

/cc: @JeremyKuhne / @merriemcgaw

@kant2002
Copy link
Contributor Author

kant2002 commented May 2, 2022

I think this is #1251 and #6267 , so my search foo was broken when I wrote this issue :(

@dreddy-work
Copy link
Member

@kant2002 , We would be tracking this with epic #6267 and closing this as duplicate.

@RussKie RussKie removed the untriaged The team needs to look at this issue in the next triage label May 5, 2022
@ghost ghost locked as resolved and limited conversation to collaborators Jun 5, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
api-suggestion (1) Early API idea and discussion, it is NOT ready for implementation
Projects
None yet
Development

No branches or pull requests

3 participants