Bug #16311: A generic method is declared as invalid for classic/linux agent if it contais curly braces with whitspace in it - Rudder - Issue Tracker

Actions

Copy link

Bug #16311

closed

A generic method is declared as invalid for classic/linux agent if it contais curly braces with whitspace in it

Added by Vincent MEMBRÉ almost 5 years ago. Updated over 1 year ago.

Status:

Rejected

Priority:

N/A

Assignee:

Vincent MEMBRÉ

Category:

Web - Technique editor

Target version:

old 6.1 issues to relocate

Pull Request:

Severity:

Major - prevents use of part of Rudder | no simple workaround

UX impact:

User visibility:

Operational - other Techniques | Rudder settings | Plugins

Effort required:

Priority:

Name check:

To do

Fix check:

To do

Regression:

Description

When you create a generic method, it will be declared as not valid for cfengine agent if it contains a coupleof curly braces with only whitespaces or nothing in between.

This syntax is used to declared empty lists in CFEngine and can also appear in strings (obviously).

The method will be displayed as non compatible with classic agent in the technique editor, will be applied correclty by a directly, But reporting will be wrong, you will get an n/a report stating that the method is not valid for classic agent, breaking reporting in Rudder (two reports made, one ok, and one n/a

As a workaround empty list can be declared using the following syntax

"my_slist" slist => splitstring("","",0);

Actions

Copy link

Updated by Vincent MEMBRÉ almost 5 years ago

Problem is when we look if a method is invalid we try to guess that we have an empty cfengine file like this

<Bunch of metadata>
bundle agent my_method (params) {

}

by looking if we match an pair of culry braces with only whitespace characters in between ({\s*}) which can happen obsiously

Actions

Copy link

Updated by Vincent MEMBRÉ almost 5 years ago

Target version changed from 5.0.15 to 5.0.16

Actions

Copy link

Updated by Alexis Mousset almost 5 years ago

Target version changed from 5.0.16 to 5.0.17

Actions

Copy link

Updated by Vincent MEMBRÉ over 4 years ago

Target version changed from 5.0.17 to 5.0.18

Actions

Copy link

Updated by François ARMAND over 4 years ago

Severity set to Major - prevents use of part of Rudder | no simple workaround
User visibility set to Operational - other Techniques | Rudder settings | Plugins
Priority changed from 0 to 47