Redaction logic causing malformed data collection payload - Known Error

Redaction logic causing malformed data collection payload Description In an effort to address security concerns around sensitive information in ACC payloads, we now check for common ways it might be found and redact the sensitive information. This led to potentiality of breaking JSON parsing for data collection.

Steps to Reproduce

Steps are going to be not very reproducible.

In the output of running_processes.rb, we had a process like this: {"running_processes":{"processes":[{"pid":"2074","ppid":"1550","command":"ibus-daemon","parameters":"--panel disable -r --xim","name":"ibus-daemon"}],"error_msg":""}}

The redaction logic matched on the --p but replaced the payload to be like so: {"running_processes":{"processes":[{"pid":"2074","ppid":"1550","command":"ibus-daemon","parameters":REDACTED disable -r --xim","name":"ibus-daemon"}],"error_msg":""}}

Note the missing opening quotation before the REDACTED string

expected: the resulting redacted json payload should not be malformed

Workaround The workaround is to add this line to MainDiscoveryHandler script include (or any payload parsing script include before the JSON.parse function is called).

In the function

addDataIntoCMDB

change try { var data = JSON.parse(result); To be try { var data = JSON.parse(result.replace(/:REDACTED/g, ':\"REDACTED'));

This fixes the broken JSON parsing by re-adding back the quotation marks. Related Problem: PRB2004499