Internal parsed entities in the internal DTD subset of an SVG content document

[murata2makoto]

I wrote:

Meanwhile, nothing in EPUB 3.2 prevents authors from defining internal parsed entities in internal DTD subsets.

In my understanding, this XML document is an SVG content document.
Here desc is an internal parsed entity declared in an internal DTD subset.
Internal parsed entities are useful for repeating something many times.

Do people agree that this is an SVG content document, as specified in EPUB 3.0, 3.0.1, and
3.2? And 3.3?

<?xml version="1.0" encoding="UTF-8"?>
<!DOCTYPE svg [
<!ENTITY desc "<desc></desc>">]>
<svg xmlns="http://www.w3.org/2000/svg" width="100%" height="100%">
    &desc;
    <g alignment-baseline="baseline"></g>
</svg>

[llemeurfr]

I agree. There is no access to the Web to resolve that, therefore it is acceptable.

[murata2makoto]

If I were a publisher, I might use this trick heavily if a long string is repeated many times.

[murata2makoto]

Are we ready to agree that all versions of EPUB 3 allow such documents as SVG content documents and close this issue?

[iherman]

This issue was discussed in a meeting.

• RESOLVED: Merge PR #1368 to address outstanding DTD issues, and close GH issues 1369-1373

View the transcript

Wendy Reid: we had resolutions at the F2F, and further discussions on github
… and came to a happy place
Matt Garrish: #1368
Matt Garrish: where we ended up was…
… we put in an allowance for a specific set of external identifiers that we have put in an appendix
… we have SVG and MathML that are allowed to be used in content docs or in separate files
… and we made a restriction against external entities in the internal DTD subset
… so it prevents some security issues but eases authoring
… so we’ll no longer force people to remove SVG DTDs from tool-generated files
… I’m hoping this is it :)
Ivan Herman: tech comment
… in fact, the changes are such that
… makes possible something that I’m not sure we really use
… I can define as part of an internal entity something that won’t go out to the network
… I’m not sure if this feature is in use
… formal comment
… there was a formal resolution on the previous version; this PR slightly changes that
… can we get a formal resolution to merge, and also close a bunch of issues which were examples of the problem?
Proposed resolution: Merge PR #1368 to address outstanding DTD issues, and close GH issues 1369-1373 (Wendy Reid)
Garth Conboy: +1
Matt Garrish: +1
Ivan Herman: +1
Charles LaPierre: +1
Matthew Chan: +1
Wendy Reid: +1
Brady Duga: +1
George Kerscher: +1
Laura Brady: +1
Bill Kasdorf: +1
Ben Schroeter: +1
Resolution #1: Merge PR #1368 to address outstanding DTD issues, and close GH issues 1369-1373

[iherman]

@wareid I believe this should be closed, too