radicallyopensecurity / pentext

PenText system: Easily create beautifully looking penetration test quotes, reports, and documents in many formats (PDF, text, JSON, CSV, ...)
GNU General Public License v2.0
117 stars 38 forks source link

removed empty <p> at start of tech descriptions #66

Closed skyanth closed 4 years ago

skyanth commented 5 years ago

Technical descriptions always started with an empty <p></p>, to the chagrin of pentesters. This commit fixes that, but I do wonder... I had expected just removing the + in line 135 would have been enough, since that would replace item.technicaldescription with the note.body but apparently it wasn't (tested it). Setting self.technicaldescription to an empty string on Finding init did the trick, but I don't understand why this is needed. Can you enlighten me, @PeterMosmans ?