Bug #18856
Updated by Vincent MEMBRÉ over 3 years ago
when setting an ldap attribute to empty string it means that we will remove it, hence when we compute a diff from the ldap modification diff we got a null for the empty value instead of an empty string. That's a quite huge problem in our code since we hope to get string from this function and we now got a null instead of an empty string. This leads to an exception when we save the diff of the element, since we expect a non null string, and having a null breaks some expectation. IN 5.0, this means there is no event log produced for change. In 6.2 it means that the change does not appear in the event log (sometimes meaning no changes at all) We shoud get an empty string instead of a string This was silenced in most cases, because we don't check when an event log is saved or not, we continue even if it fails. But in sme api we look for this event log, and don't ignore the result of the call. small example of the behavior in 5.0, we make two modification of the same directive, one adding description, one removing it. We only got thr first event log, adding the description !https://issues.rudder.io/attachments/download/1873/5.0%20-%20event%20log.gif! !5.0 - event log.gif!