Bug #25704
openevent log detail show shouldBeReplacedByDiff
Description
A node modified diff displays shouldBeReplacedByDiff
Files
Updated by Vincent MEMBRÉ 9 months ago
- Target version changed from 8.2.0 to 8.2.1
Updated by Vincent MEMBRÉ 8 months ago
- Target version changed from 8.2.1 to 8.2.2
Updated by Nicolas CHARLES 8 months ago
- Assignee set to François ARMAND
- Priority changed from N/A to 2
Exists also probably in 8.1
to reproduce: have a node with property, send an inventory with node inventory properties, you get this result
Updated by Vincent MEMBRÉ 8 months ago
- Target version changed from 8.2.2 to 8.2.3
Updated by François ARMAND 8 months ago
So, the problem is that storage.save(updated)
is lying:
- it always return inventory properties in the new part, but not in the old node part of the event,
- it does not return the change in rudder settings: it seens in both case the new value.
Updated by François ARMAND 8 months ago
- Target version changed from 8.2.3 to 8.1.10
It happens in 8.1 too.
Updated by François ARMAND 8 months ago
- File clipboard-202411281718-svnul.png clipboard-202411281718-svnul.png added
- File clipboard-202411281720-babn0.png clipboard-202411281720-babn0.png added
"updated" and "s.newNode" have the correct values:
but s.oldNode doesn't:
Updated by Vincent MEMBRÉ 7 months ago
- Target version changed from 8.1.10 to 8.1.11
Updated by Vincent MEMBRÉ 6 months ago
- Target version changed from 8.1.11 to 8.1.12
Updated by Vincent MEMBRÉ 5 months ago
- Target version changed from 8.1.12 to 8.1.13
Updated by Vincent MEMBRÉ 2 months ago
- Target version changed from 8.1.13 to 8.2.6
Updated by Vincent MEMBRÉ 2 months ago
- Target version changed from 8.2.6 to 8.2.7
Updated by Vincent MEMBRÉ about 1 month ago
- Target version changed from 8.2.7 to 8.2.8
Updated by Nicolas CHARLES 9 days ago
I'm having these error in 8.3, might be related
Updated by Clark ANDRIANASOLO 9 days ago
If it is only a display issue, it looks like a duplicate (special case) of #27114, the diff for node properties is indeed using the javascript library jsondiffpatch
but it cannot be called due to strict CSP headers
Updated by Clark ANDRIANASOLO 9 days ago
- Related to Bug #27114: Many diffs no longer display in event logs added
Updated by Nicolas CHARLES 9 days ago
- Related to Bug #27185: CSP error in change logs pages added