Bug #20415: The technique editor allows saving methods with missing parameters - Rudder - Issue Tracker

Actions

Copy link

Bug #20415

closed

The technique editor allows saving methods with missing parameters

Added by Alexis Mousset almost 3 years ago. Updated almost 3 years ago.

Status:

Released

Priority:

N/A

Assignee:

Nicolas CHARLES

Category:

Web - Technique editor

Target version:

7.0.0~rc3

Pull Request:

https://github.com/Normation/rudder/p...

Severity:

UX impact:

User visibility:

Effort required:

Priority:

Name check:

To do

Fix check:

To do

Regression:

Description

This leads to useless error messages for the user:

Error when saving technique: The server had a problem, try again later

and an error log in the webapp:

[2021-12-14 13:03:18+0000] ERROR techniques.writer - An error occurred when compiling technique 'Inventory hook for updates' (id : 'BundleName(Inventory_hook_for_updates)') with rudderc, error details in /var/log/rudder/rudderc/failures/2021-12-14T13:03:18.901Z_Inventory_hook_for_updates.log, falling back to old saving process
Fiber failed.
An unchecked error was produced.
com.normation.cfclerk.xmlparsers.EmptyReportKeysValue: In 'expectedReportKey File from local source', the element REPORTKEYS must have a non empty list of provided values: <REPORTKEYS><VALUE>val foo</VALUE><VALUE>...
    at com.normation.cfclerk.domain.SectionVariableSpec$.apply(VariableAndSectionSpec.scala:410)
    at com.normation.cfclerk.xmlparsers.VariableSpecParser.parseSectionVariableSpec(VariableSpecParser.scala:134)
    at com.normation.cfclerk.xmlparsers.SectionSpecParser.parseOneVariable$1(SectionSpecParser.scala:192)
    at com.normation.cfclerk.xmlparsers.SectionSpecParser.$anonfun$parseChildren$4(SectionSpecParser.scala:205)
    at cats.data.Chain$.$anonfun$traverseViaChain$1(Chain.scala:797)
    at cats.Later.value$lzycompute(Eval.scala:148)