Project

General

Profile

Actions

Bug #20142

open

Invalid metadata (section with empty name) generated by the technique editor

Added by Félix DALLIDET about 3 years ago. Updated 6 months ago.

Status:
New
Priority:
N/A
Assignee:
-
Category:
Web - Technique editor
Target version:
Severity:
Critical - prevents main use of Rudder | no workaround | data loss | security
UX impact:
User visibility:
Operational - other Techniques | Rudder settings | Plugins
Effort required:
Priority:
0
Name check:
To do
Fix check:
To do
Regression:

Description

While working on the CIS technique for rhel7 in rudder language, I ended up with a broken metadata.xml, resulting in a broken technique.
No errors where detected or thrown by the technique editor, but my technique does not appear in the directive webapp page nor in the technique tree.
The webapp logs are raising the following errors:

[2021-10-11 14:13:22+0000] ERROR application - An error occurred when compiling technique 'My technique name' (id : 'BundleName(CIS_rhel7_setup_filesystem)') with rudderc, error details in /var/log/rudder/rudderc/failures/2021-10-11T14:13:22.704Z_CIS_rhel7_setup_filesystem.log, falling back to old saving process
[2021-10-11 14:13:23+0000] ERROR techniques.reader - Error with technique at path: '/ncf_techniques/CIS_rhel7_setup_filesystem/1.0/metadata.xml', it will be ignored. Error: Chained: Couldn't parse Section in CIS_rhel7_setup_filesystem/1.0 -> My technique name for XML: <SECTION reporting="sum" name="Initial Setup" multivalued="true" component="true">
      <SECTION reporting="sum" name="Additional Process Hardening" multivalued="true" component="true">
        <SECTION reporting="sum" name="Ensure XD/NX support is enabled" multivalued="true" component="true">
          <SECTION name="Condition from command" multivalued="true" component="true">
            <REPORTKEYS>
              <VALUE>xd_nx_support_enabled</VALUE>
            </REPORTKEYS>
          </SECTION>
          <SECTION name="Report if condition" multivalued="true" component="true">
            <REPORTKEYS>
              <VALUE>Ensure core dumps are restricted</VALUE>

I could not find what the issue was, the xml seems correct (xmllint accepts it).
The metadata.xml is pretty big, and is attached in the ticket. There are two issues here:
  • The technique should not be saved and failing silently in the technique editor if the metadata.xml is incorrect
  • The metadata does not seem to work as intended

Files

metadata.xml (93.4 KB) metadata.xml Félix DALLIDET, 2021-10-18 11:09

Related issues 1 (0 open1 closed)

Related to Rudder - Bug #19930: We were able to save a technique without a name which caused chaos ResolvedActions
Actions #1

Updated by Vincent MEMBRÉ about 3 years ago

  • Target version changed from 7.0.0~beta2 to 7.0.0~beta3
Actions #2

Updated by Vincent MEMBRÉ about 3 years ago

  • Target version changed from 7.0.0~beta3 to 7.0.0~rc1
Actions #3

Updated by Vincent MEMBRÉ about 3 years ago

  • Target version changed from 7.0.0~rc1 to 7.0.0~rc2
  • Priority changed from 76 to 75
Actions #4

Updated by Vincent MEMBRÉ about 3 years ago

  • Target version changed from 7.0.0~rc2 to 7.0.0~rc3
Actions #5

Updated by Vincent MEMBRÉ almost 3 years ago

  • Target version changed from 7.0.0~rc3 to 7.0.0
  • Priority changed from 75 to 73
Actions #6

Updated by Vincent MEMBRÉ almost 3 years ago

  • Target version changed from 7.0.0 to 7.0.1
Actions #7

Updated by François ARMAND almost 3 years ago

  • Priority changed from 73 to 72

The root cause is lower in the error trace:

cause was: Parsing: Section must have name. Missing name for: <SECTION reporting="weighted" name="" multivalued="true" component="true">

So the technique editor missed generation of a name somewhere.

Actions #8

Updated by François ARMAND almost 3 years ago

  • Subject changed from Invalid metadata generated by the technique editor to Invalid metadata (section with empty name) generated by the technique editor
Actions #9

Updated by François ARMAND almost 3 years ago

  • Related to Bug #19930: We were able to save a technique without a name which caused chaos added
Actions #10

Updated by Vincent MEMBRÉ almost 3 years ago

  • Target version changed from 7.0.1 to 7.0.2
Actions #11

Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 7.0.2 to 7.0.3
  • Priority changed from 72 to 70
Actions #12

Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 7.0.3 to 7.0.4
  • Priority changed from 70 to 69
Actions #13

Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 7.0.4 to 7.0.5
  • Priority changed from 69 to 67
Actions #14

Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 7.0.5 to 7.0.6
  • Priority changed from 67 to 66
Actions #15

Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 7.0.6 to 7.0.7
  • Priority changed from 66 to 0
Actions #16

Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 7.0.7 to 7.0.8
Actions #17

Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 7.0.8 to 1002
Actions #18

Updated by Vincent MEMBRÉ about 2 years ago

  • Target version changed from 1002 to 7.1.7
Actions #19

Updated by Vincent MEMBRÉ about 2 years ago

  • Target version changed from 7.1.7 to 7.1.8
Actions #20

Updated by Vincent MEMBRÉ about 2 years ago

  • Target version changed from 7.1.8 to 7.1.9
Actions #21

Updated by Vincent MEMBRÉ about 2 years ago

  • Target version changed from 7.1.9 to 7.1.10
Actions #22

Updated by Vincent MEMBRÉ almost 2 years ago

  • Target version changed from 7.1.10 to 1016
Actions #23

Updated by Alexis Mousset almost 2 years ago

  • Target version changed from 1016 to 7.2.5
Actions #24

Updated by Vincent MEMBRÉ almost 2 years ago

  • Target version changed from 7.2.5 to 7.2.6
Actions #25

Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 7.2.6 to 7.2.7
Actions #26

Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 7.2.7 to 7.2.8
Actions #27

Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 7.2.8 to 7.2.9
Actions #28

Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 7.2.9 to 7.2.10
Actions #29

Updated by Alexis Mousset over 1 year ago

  • Target version changed from 7.2.10 to 7.2.11
Actions #30

Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 7.2.11 to 1046
Actions #31

Updated by Alexis Mousset about 1 year ago

  • Target version changed from 1046 to 7.3.8
Actions #32

Updated by Vincent MEMBRÉ about 1 year ago

  • Target version changed from 7.3.8 to 7.3.9
Actions #33

Updated by Vincent MEMBRÉ about 1 year ago

  • Target version changed from 7.3.9 to 7.3.10
Actions #34

Updated by Vincent MEMBRÉ about 1 year ago

  • Target version changed from 7.3.10 to 7.3.11
Actions #35

Updated by Vincent MEMBRÉ 11 months ago

  • Target version changed from 7.3.11 to 7.3.12
Actions #36

Updated by Vincent MEMBRÉ 10 months ago

  • Target version changed from 7.3.12 to 7.3.13
Actions #37

Updated by Vincent MEMBRÉ 10 months ago

  • Target version changed from 7.3.13 to 7.3.14
Actions #38

Updated by Vincent MEMBRÉ 8 months ago

  • Target version changed from 7.3.14 to 7.3.15
Actions #39

Updated by Vincent MEMBRÉ 7 months ago

  • Target version changed from 7.3.15 to 7.3.16
Actions #40

Updated by Vincent MEMBRÉ 6 months ago

  • Target version changed from 7.3.16 to 7.3.17
Actions

Also available in: Atom PDF