-
Bug
-
Resolution: Fixed
-
Major
On tapResults page, if the output of a given test contains HTML, it renders it out onto the tapResults report wholesale and unescaped.
If the HTML is partial, it can lead to some output being outside of a result table or missing on render entirely.
To illustrate what I mean, I've included some offending TAP file content and some screenshots of the results(masking applied by me to protect some data).
# <div id="PageWrapper"> # # <div id="Header"> # <div class="Liner"> # </div><!-- end Liner --> # </div><!-- end Header --> # <div id="ContentWrapper"> # <div id="ContentColumn"> # <div class="Liner"> # # # </div><!-- end Liner --> # </div><!-- end ContentColumn --> # </div><!-- end ContentWrapper --> # # <div id="NavColumn"> # <div class="Liner"> # </div><!-- end Liner --> # </div><!-- end NavColumn --> #
[JENKINS-17878] HTML test output in tapResults not escaped
Real ONeil
created issue -
Real ONeil
made changes -
| Description |
Original:
On tapResults page, if the output of a given test contains HTML, it renders it out onto the tapResults report wholesale and unescaped. If the HTML is partial, it can lead to some output being outside of a result table or missing on render entirely. To illustrate what I mean, I've included some offending TAP file content and some screenshots of the results(masking applied by me to protect some data). {code} # <div id="PageWrapper"> # # <div id="Header"> # <div class="Liner"> # </div><!-- end Liner --> # </div><!-- end Header --> # <div id="ContentWrapper"> # <div id="ContentColumn"> # <div class="Liner"> # # # </div><!-- end Liner --> # </div><!-- end ContentColumn --> # </div><!-- end ContentWrapper --> # # <div id="NavColumn"> # <div class="Liner"> # </div><!-- end Liner --> # </div><!-- end NavColumn --> # {/code} |
New:
On tapResults page, if the output of a given test contains HTML, it renders it out onto the tapResults report wholesale and unescaped. If the HTML is partial, it can lead to some output being outside of a result table or missing on render entirely. To illustrate what I mean, I've included some offending TAP file content and some screenshots of the results(masking applied by me to protect some data). {code} # <div id="PageWrapper"> # # <div id="Header"> # <div class="Liner"> # </div><!-- end Liner --> # </div><!-- end Header --> # <div id="ContentWrapper"> # <div id="ContentColumn"> # <div class="Liner"> # # # </div><!-- end Liner --> # </div><!-- end ContentColumn --> # </div><!-- end ContentWrapper --> # # <div id="NavColumn"> # <div class="Liner"> # </div><!-- end Liner --> # </div><!-- end NavColumn --> # {code} |
| Status | Original: Open [ 1 ] | New: In Progress [ 3 ] |
| Status | Original: In Progress [ 3 ] | New: Open [ 1 ] |
| Resolution | New: Fixed [ 1 ] | |
| Status | Original: Open [ 1 ] | New: Resolved [ 5 ] |
| Status | Original: Resolved [ 5 ] | New: Closed [ 6 ] |
| Workflow | Original: JNJira [ 149174 ] | New: JNJira + In-Review [ 206633 ] |