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

Fix: XmlWriterSettings.Encoding must be registered in static Encoding #56164

Merged
merged 2 commits into from
Jul 23, 2021

Conversation

krwq
Copy link
Member

@krwq krwq commented Jul 22, 2021

Fixes: #32164

Bring back full framework behavior to avoid throwing errors when encoding is not registered or custom encoding is used.

https://github.com/microsoft/referencesource/blob/master/System.Xml/System/Xml/Core/XmlEncodedRawTextWriter.cs#L177-L178

@ghost
Copy link

ghost commented Jul 22, 2021

Tagging subscribers to this area: @buyaa-n, @krwq
See info in area-owners.md if you want to be subscribed.

Issue Details

Fixes: #32164

Bring back full framework behavior to avoid throwing errors when encoding is not registered or custom encoding is used.

https://github.com/microsoft/referencesource/blob/master/System.Xml/System/Xml/Core/XmlEncodedRawTextWriter.cs#L177-L178

Author: krwq
Assignees: -
Labels:

area-System.Xml

Milestone: -

Copy link
Contributor

@buyaa-n buyaa-n left a comment

Choose a reason for hiding this comment

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

LGTM

…dRawTextWriter.cs

Co-authored-by: Buyaa Namnan <bunamnan@microsoft.com>
@krwq
Copy link
Member Author

krwq commented Jul 23, 2021

The failures seem unrelated. Merging

@krwq krwq merged commit fdead97 into main Jul 23, 2021
@akoeplinger akoeplinger deleted the krwq-patch-1 branch July 26, 2021 10:15
@ghost ghost locked as resolved and limited conversation to collaborators Aug 25, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

XmlWriterSettings.Encoding must be registered in static Encoding
3 participants