resolves #4698

[wdower]

The fast-xml-parser module was set to strip out the namespace prefix from XML tags (ex the cm in <cm:compliance-check-name) using the removeNSPrefix option for parseXML.

Problem was that the nessus mapper was looking for XML tags that had not had the namespace stripped out. So parseXML was (correctly) returning values like compliance-check-name as attributes, but the mapper was looking for cm:compliance-check-name.

[georgedias]

This was a big oversight, given that the fast-xml-parser was striping the attributes prefixes, great catch.

I would make use of const for the attributes being searched, there is, use:

const COMPLIANCE_CHEK_NAME = 'compliance-check-name';
const COMPLIANCE_INFO = 'compliance-info';
const COMPLIANCE_SOLUTION = 'compliance-solution';
const COMPLIANCE_RESULT = 'compliance-result';
const COMPLIANCE_ACTUAL_VALUE = 'compliance-actual-value';

[georgedias]

Thanks for the updates

[Amndeep7]

If we can get an anonymized sample file that would test against this, it would be great. Can you ask your sponsor to see if they would mind if we attempted to anonymize their data? Doesn't need to be a blocker for this pr.

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication