Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-17759

Email notification source tries to indent HTML from console output

    XMLWordPrintable

    Details

    • Similar Issues:

      Description

      We use the extended email notification plugin and its regex tokens to email a section of the console output. That section is a basic HTML with a few <pre> tags that are pre formatted. After the latest update of both the plugin and jenkins, the section of the console output started getting a "tidyhtml" treatment.

      Old email source and console output looked like this:
      <html>
      <head></head>
      <body>
      <table border="1">
      <tbody>
      <tr>
      <td> <b>TEXT</b> </td>
      <td> <b>TEXT</b><pre>
      line
      line<v1>line
      line<v1>line
      ...

      New email source looks like this (and messing up the actual email):
      <html>
      <head></head>
      <body>
      <table border="1">
      <tbody>
      <tr>
      <td> <b>TEXT</b> </td>
      <td> <b>TEXT</b><pre>
      line
      line
      <v1>
      line
      line
      <v1>
      line
      ...

      While I don't mind indentation and pretty printing, it seems like a bug to try to beautify (by introducing new lines and spaces) the output of a console.

      EDIT: alright, looks like jira clears any leading blanks so I'll use * to indicate space.
      New output:
      <html>
      *<head></head>
      **<body>
      ***<table border="1">
      ***<tbody>
      ****<tr>
      *****<td> <b>TEXT</b> </td>
      *****<td> <b>TEXT</b><pre>
      line
      line
      ******<v1>
      *******line
      line
      *******<v1>
      ********line

        Attachments

          Activity

          Hide
          slide_o_mix Alex Earl added a comment -

          My guess is that this is from the cssinliner. I'll take a look

          Show
          slide_o_mix Alex Earl added a comment - My guess is that this is from the cssinliner. I'll take a look
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Alex Earl
          Path:
          src/main/java/hudson/plugins/emailext/plugins/CssInliner.java
          src/test/java/hudson/plugins/emailext/plugins/CssInlinerTest.java
          http://jenkins-ci.org/commit/email-ext-plugin/fd016daf6695875f03320474848000201a4409c8
          Log:
          Fixed JENKINS-17759

          Turned off pretty printing in the CssInliner.

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Alex Earl Path: src/main/java/hudson/plugins/emailext/plugins/CssInliner.java src/test/java/hudson/plugins/emailext/plugins/CssInlinerTest.java http://jenkins-ci.org/commit/email-ext-plugin/fd016daf6695875f03320474848000201a4409c8 Log: Fixed JENKINS-17759 Turned off pretty printing in the CssInliner.
          Hide
          slide_o_mix Alex Earl added a comment -

          Turned off pretty printing in the css inliner (jsoup).

          Show
          slide_o_mix Alex Earl added a comment - Turned off pretty printing in the css inliner (jsoup).
          Hide
          todorovge Georgi Todorov added a comment -

          Thanks!

          Show
          todorovge Georgi Todorov added a comment - Thanks!

            People

            • Assignee:
              slide_o_mix Alex Earl
              Reporter:
              todorovge Georgi Todorov
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: