Bogus xmlns:xlink attribute leaked into HTML output when <index> was used.

[anarazel]

Hi,

In postgres' sgml docs I noticed 100kB+ of xmlns:xlink=..., which turns out to be caused by autoidx.xsl not including xlink in exclude-result-prefixes. Previously reported at https://sourceforge.net/p/docbook/bugs/1384/

Because postgres customizes the bookindex output, including outputting the <div class="index"> , the xmlns:xlink gets pushed down into the elements that autoidx emits. Leading to output like this:

        <div class="indexdiv" id="indexdiv-A">
          <h3>A</h3>
          <dl>
            <dt xmlns:xlink="http://www.w3.org/1999/xlink" id="ientry-idm19499">abbrev, <a class="indexterm" href="functio
ns-net.html">Network Address Functions and Operators</a></dt>
            <dt xmlns:xlink="http://www.w3.org/1999/xlink" id="ientry-idm86953">ABORT, <a class="indexterm" href="sql-abor
t.html">ABORT</a></dt>
            <dt xmlns:xlink="http://www.w3.org/1999/xlink" id="ientry-idm11113">abs, <a class="indexterm" href="functions-
math.html">Mathematical Functions and Operators</a></dt>
            <dt xmlns:xlink="http://www.w3.org/1999/xlink" id="ientry-idm3024">ACL, <a class="indexterm" href="ddl-priv.ht
ml">Privileges</a></dt>

Not fun :)

Should I open a pull request with the obvious change? Or would you prefer to take it from https://sourceforge.net/p/docbook/bugs/1384/?

Regards,

Andres

[bobstayton]

Go ahead and create a pull request.  Thanks. Bob Stayton ***@***.***

…

On 2/13/2022 12:39 PM, Andres Freund wrote: Hi, In postgres' sgml docs I noticed 100kB+ of |xmlns:xlink=...|, which turns out to be caused by autoidx.xsl not including xlink in |exclude-result-prefixes|. Previously reported at https://sourceforge.net/p/docbook/bugs/1384/ Because postgres customizes the bookindex output, including outputting the |<div class="index">| , the |xmlns:xlink| gets pushed down into the elements that autoidx emits. Leading to output like this: <div class="indexdiv"id="indexdiv-A"> <h3>A</h3> <dl> <dt xmlns:xlink="http://www.w3.org/1999/xlink"id="ientry-idm19499">abbrev,<a class="indexterm"href="functio ns-net.html">Network Address Functions and Operators</a></dt> <dt xmlns:xlink="http://www.w3.org/1999/xlink"id="ientry-idm86953">ABORT,<a class="indexterm"href="sql-abor t.html">ABORT</a></dt> <dt xmlns:xlink="http://www.w3.org/1999/xlink"id="ientry-idm11113">abs,<a class="indexterm"href="functions- math.html">Mathematical Functions and Operators</a></dt> <dt xmlns:xlink="http://www.w3.org/1999/xlink"id="ientry-idm3024">ACL,<a class="indexterm"href="ddl-priv.ht ml">Privileges</a></dt> Not fun :) Should I open a pull request with the obvious change? Or would you prefer to take it from https://sourceforge.net/p/docbook/bugs/1384/? Regards, Andres — Reply to this email directly, view it on GitHub <#239>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AC6W3QMJJEQUBYGMTMQZI43U3AJJ3ANCNFSM5OJVFAQA>. Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>. You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>