If I look into the code I can see in WriterAppender writeFooter

String h = layout.getFileFooter();
if ((h != null) && (this.writer != null)) {
try {
this.writer.write(h);
String pf = layout.getPresentationFooter();
if (pf != null)

{ this.writer.write(pf); }

And if I look into HTMLLayoutBase I see

getFileFooter => </body></html>
getPresentationFooter => </table>

which means we are first closing the HTML and then the TABLE?

Fix:

String pf = layout.getPresentationFooter();
if ((pf != null) && (this.writer != null)) {
try {
this.writer.write(pf);
String h = layout.getFileFooter();
if (h != null)

{ this.writer.write(h); }