Project

General

Profile

Actions

Bug #27114

open

Many diffs no longer display in event logs

Added by Clark ANDRIANASOLO about 18 hours ago. Updated about 16 hours ago.

Status:
New
Priority:
To review
Category:
Web - Config management
Target version:
Severity:
Minor - inconvenience | misleading | easy workaround
UX impact:
I dislike using that feature
User visibility:
Getting started - demo | first install | Technique editor and level 1 Techniques
Effort required:
Medium
Priority:
58
Name check:
To do
Fix check:
To do
Regression:
No

Description

it seems that many JS code that are executed as AJAX in the Event logs (also the change logs page in starting from 8.3) are not executed.
We use diff libraries and we inject JS code that does not seem to be executed at all, makeDiff and jsondiffpatch
So, we can no longer see the diff of tags for directives/rules (see #15019), or JSON semantic diff for node properties.

Screenshot for a missing directive diff on tags

There is a CSP violation in the page which could be the cause of that.


Files

clipboard-202506171645-uib6i.png (264 KB) clipboard-202506171645-uib6i.png Clark ANDRIANASOLO, 2025-06-17 16:45

Related issues 1 (0 open1 closed)

Related to Rudder - Bug #25352: Event log rollback action is prevented from CSP headersReleasedRaphael GAUTHIERActions
Actions #1

Updated by Clark ANDRIANASOLO about 18 hours ago

  • Description updated (diff)
Actions #2

Updated by Clark ANDRIANASOLO about 16 hours ago

  • Effort required changed from Very Small to Medium
  • Priority changed from 103 to 58

The JS queries the event logs endpoint which returns JS code in some field, but in #25352 we made assumptions about the response of the API, which may only contain JS, but no HTML.

So we should find a way to run diffs, we can not longer run script tags from the server, it will be far from trivial

Actions #3

Updated by Clark ANDRIANASOLO about 16 hours ago

  • Related to Bug #25352: Event log rollback action is prevented from CSP headers added
Actions

Also available in: Atom PDF