[PR #12] [MERGED] Ignore DTD when reading XML PList documents #47

New Issue

claunia · 2026-01-30T21:26:53Z

claunia commented

2026-01-30 21:26:53 +00:00

📋 Pull Request Information

Original PR: https://github.com/claunia/plist-cil/pull/12
Author: @qmfrederik
Created: 6/28/2016
Status: ✅ Merged
Merged: 7/9/2016
Merged by: @claunia

Base: master ← Head: fixes/dtd-processing

📝 Commits (1)

fb7e431 Ignore DTD when reading XML PList documents

📊 Changes

1 file changed (+8 additions, -1 deletions)

View changed files

📝 plist-cil/XmlPropertyListParser.cs (+8 -1)

📄 Description

I'm slightly surprised this wasn't an issue on .NET 4.6, but when trying to read a XmlDocument which contains a DTD definition (like the XML property lists), .NET Core will throw an exception unless you've instructed it to either ignore or validate DTD's.

This PR causes .NET to ignore them (they are not used anyway).

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/claunia/plist-cil/pull/12 **Author:** [@qmfrederik](https://github.com/qmfrederik) **Created:** 6/28/2016 **Status:** ✅ Merged **Merged:** 7/9/2016 **Merged by:** [@claunia](https://github.com/claunia) **Base:** `master` ← **Head:** `fixes/dtd-processing` --- ### 📝 Commits (1) - [`fb7e431`](https://github.com/claunia/plist-cil/commit/fb7e431ac8d20db2cf9f6852b4a1fc5de77f3ce5) Ignore DTD when reading XML PList documents ### 📊 Changes **1 file changed** (+8 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `plist-cil/XmlPropertyListParser.cs` (+8 -1) </details> ### 📄 Description I'm slightly surprised this wasn't an issue on .NET 4.6, but when trying to read a XmlDocument which contains a DTD definition (like the XML property lists), .NET Core will throw an exception unless you've instructed it to either ignore or validate DTD's. This PR causes .NET to ignore them (they are not used anyway). --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

claunia added the pull-request label 2026-01-30 21:26:53 +00:00

claunia closed this issue

2026-01-30 21:26:55 +00:00

claunia referenced this issue

2026-01-30 21:27:02 +00:00

[PR #47] [MERGED] Consider booleans to be serialization 'primitives' as well #77

claunia referenced this issue

2026-01-30 21:27:02 +00:00

[PR #47] Consider booleans to be serialization 'primitives' as well #82

Sign in to join this conversation.