Skip to content

Fix: use pretty status names in summary tables + some code cleanup #167

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
PeterMosmans merged 2 commits into from
Aug 19, 2026
Merged

Fix: use pretty status names in summary tables + some code cleanup #167

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
b31a73e
Fix: use pretty status names in summary tables + some code cleanup
Synchro Aug 18, 2026
ee1195a
Fix: Conclusion copy
Synchro Aug 19, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 5 additions & 7 deletions source/conclusion.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,17 +2,15 @@
<section id="conclusion" xml:base="conclusion.xml" break="before" inexecsummary="yes">
<title>Conclusion</title>
<p>We discovered <finding_count/>-severity issues during this penetration test.</p>

<p><todo desc="Add some paragraphs describing in high level the impact of these findings on the organization, priorities in fixing them"/></p>

<p>
<todo desc="Add some paragraphs describing in high level the impact of these findings on the organization, priorities in fixing them"/>
</p>
<p>We recommend fixing all of the issues found and then performing a retest in order to ensure
that mitigations are effective and that no new vulnerabilities have been introduced.</p>
<p>Finally, we want to emphasize that security is a process – this penetration test is just a
one-time snapshot. Security posture must be continuously evaluated and improved. Regular audits
and ongoing improvements are essential in order to maintain control of your corporate
<p>Finally, we want to emphasize that security is a process that must be continuously evaluated and improved – this penetration test is just a one-time snapshot.
Regular audits and ongoing improvements are essential in order to maintain control of your
information security. We hope that this pentest report (and the detailed explanations of our
findings) will contribute meaningfully towards that end.</p>
<p>Please don't hesitate to let us know if you have any further questions, or need further
clarification on anything in this report.</p>

</section>
99 changes: 37 additions & 62 deletions xslt/auto.xslt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,9 @@
<xsl:stylesheet
xmlns:xsl="http://www.w3.org/1999/XSL/Transform"
xmlns:xs="http://www.w3.org/2001/XMLSchema"
exclude-result-prefixes="xs"
exclude-result-prefixes="xs my"
xmlns:fo="http://www.w3.org/1999/XSL/Format"
xmlns:my="http://www.radical.sexy"
version="2.0"
>

Expand Down Expand Up @@ -146,10 +147,7 @@
</xsl:template>

<xsl:template name="findingsSummaryContent">
<fo:table-row
xsl:use-attribute-sets="TableFont"
keep-together.within-column="always"
>
<fo:table-row xsl:use-attribute-sets="TableFont" keep-together.within-column="always">
<xsl:if test="position() mod 2 != 0">
<xsl:attribute name="background-color">#ededed</xsl:attribute>
</xsl:if>
Expand Down Expand Up @@ -185,40 +183,36 @@

<!-- Status Section -->
<xsl:if test="@status and @status != 'none' and @status != ''">

<xsl:variable name="prettyStatus">
<xsl:sequence
select="string-join(for $x in tokenize(@status, '_') return my:titleCase($x), ' ')"
/>
</xsl:variable>
<fo:block>
<fo:inline
xsl:use-attribute-sets="bold"
>Status: </fo:inline>
<fo:inline xsl:use-attribute-sets="bold">Status: </fo:inline>
<xsl:choose>
<xsl:when
test="@status = 'new' or @status = 'unresolved'"
>
<xsl:when test="@status = 'new' or @status = 'unresolved'">
<fo:inline>
<xsl:attribute name="color">
<xsl:value-of select="$color_new" />
</xsl:attribute>
<xsl:value-of select="@status" />
<xsl:value-of select="$prettyStatus" />
</fo:inline>
</xsl:when>
<xsl:when test="@status = 'not_retested'">
<fo:inline>
<xsl:attribute name="color">
<xsl:value-of
select="$color_notretested"
/>
<xsl:value-of select="$color_notretested"/>
</xsl:attribute>
<xsl:value-of select="@status" />
<xsl:value-of select="$prettyStatus" />
</fo:inline>
</xsl:when>
<xsl:when test="@status = 'resolved'">
<fo:inline>
<xsl:attribute name="color">
<xsl:value-of
select="$color_resolved"
/>
<xsl:value-of select="$color_resolved"/>
</xsl:attribute>
<xsl:value-of select="@status" />
<xsl:value-of select="$prettyStatus" />
</fo:inline>
</xsl:when>
<xsl:otherwise>
Expand Down Expand Up @@ -342,10 +336,7 @@
</xsl:template>

<xsl:template name="recommendationsSummaryContent">
<fo:table-row
xsl:use-attribute-sets="TableFont"
keep-together.within-column="always"
>
<fo:table-row xsl:use-attribute-sets="TableFont" keep-together.within-column="always">
<xsl:if test="position() mod 2 != 0">
<xsl:attribute name="background-color">#ededed</xsl:attribute>
</xsl:if>
Expand All @@ -371,55 +362,50 @@
<xsl:value-of select="@threatLevel" />
</xsl:when>
</xsl:choose>
</fo:inline>
</fo:inline>
</fo:block>
</xsl:if>

<!-- Type -->
<xsl:if test="@type">
<fo:block>
<fo:inline
xsl:use-attribute-sets="bold"
>Type: </fo:inline>
<fo:inline xsl:use-attribute-sets="bold">Type: </fo:inline>
<xsl:value-of select="@type" />
</fo:block>
</xsl:if>

<!-- Status -->
<xsl:if test="@status and @status != 'none'">
<!-- Status Section -->
<xsl:if test="@status and @status != 'none' and @status != ''">
<xsl:variable name="prettyStatus">
<xsl:sequence
select="string-join(for $x in tokenize(@status, '_') return my:titleCase($x), ' ')"
/>
</xsl:variable>
<fo:block>
<fo:inline
xsl:use-attribute-sets="bold"
>Status: </fo:inline>
<fo:inline xsl:use-attribute-sets="bold">Status: </fo:inline>
<xsl:choose>
<xsl:when
test="@status = 'new' or @status = 'unresolved'"
>
<xsl:when test="@status = 'new' or @status = 'unresolved'">
<fo:inline>
<xsl:attribute name="color">
<xsl:value-of select="$color_new" />
</xsl:attribute>
<xsl:value-of select="@status" />
<xsl:value-of select="$prettyStatus" />
</fo:inline>
</xsl:when>
<xsl:when test="@status = 'not_retested'">
<fo:inline>
<xsl:attribute name="color">
<xsl:value-of
select="$color_notretested"
/>
<xsl:value-of select="$color_notretested"/>
</xsl:attribute>
<xsl:value-of select="@status" />
<xsl:value-of select="$prettyStatus" />
</fo:inline>
</xsl:when>
<xsl:when test="@status = 'resolved'">
<fo:inline>
<xsl:attribute name="color">
<xsl:value-of
select="$color_resolved"
/>
<xsl:value-of select="$color_resolved"/>
</xsl:attribute>
<xsl:value-of select="@status" />
<xsl:value-of select="$prettyStatus"/>
</fo:inline>
</xsl:when>
<xsl:otherwise>
Expand All @@ -437,19 +423,14 @@
<xsl:for-each select="labels/label">
<fo:inline xsl:use-attribute-sets="label">
<xsl:attribute name="background-color">
<xsl:value-of
select="/pentest_report/meta/labels/label[@name = current()]/@color"
/>
<xsl:value-of select="/pentest_report/meta/labels/label[@name = current()]/@color"/>
</xsl:attribute>
<xsl:attribute name="color">
<xsl:value-of
select="/pentest_report/meta/labels/label[@name = current()]/@text"
/>
<xsl:value-of select="/pentest_report/meta/labels/label[@name = current()]/@text"/>
</xsl:attribute>
<xsl:value-of select="." />
</fo:inline>
<xsl:text
> </xsl:text> <!-- Add space between labels -->
<xsl:text> </xsl:text> <!-- Add space between labels -->
</xsl:for-each>
</fo:block>
</xsl:if>
Expand All @@ -462,16 +443,10 @@
<!-- Check for recommendation_summary first -->
<xsl:choose>
<xsl:when test="recommendation_summary">
<xsl:apply-templates
select="recommendation_summary"
mode="summarytable"
/>
<xsl:apply-templates select="recommendation_summary" mode="summarytable"/>
</xsl:when>
<xsl:otherwise>
<xsl:apply-templates
select="recommendation"
mode="summarytable"
/>
<xsl:apply-templates select="recommendation" mode="summarytable"/>
</xsl:otherwise>
</xsl:choose>
</fo:block>
Expand Down
2 changes: 1 addition & 1 deletion xslt/findings.xslt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -175,7 +175,7 @@
Update
</fo:inline>
</xsl:when>
</xsl:choose>
</xsl:choose>
<xsl:if test="@date">
<xsl:text> </xsl:text>
<fo:inline xsl:use-attribute-sets="status-tag">
Expand Down
Loading