Project

General

Profile

Actions

Bug #12269

open

Regex in metadata.xml don't match when using .

Added by Benoît PECCATTE almost 7 years ago. Updated 20 days ago.

Status:
New
Priority:
N/A
Assignee:
-
Category:
Web - Config management
Target version:
Severity:
Minor - inconvenience | misleading | easy workaround
UX impact:
User visibility:
Infrequent - complex configurations | third party integrations
Effort required:
Very Small
Priority:
0
Name check:
Fix check:
Regression:
No

Description

Regex on input in metadata.xml are used to match multiline strings.
In this case, we expect to use the regex to match the whole input, so '^' '$' should match start and end of string and '.' should match newlines.

This means the regex should use both 'single line' (aka s) and 'multiple lines' (aka m) options.


Related issues 1 (0 open1 closed)

Related to Rudder - Bug #12263: Wrong regex in FileTemplate techniqueReleasedBenoît PECCATTEActions
Actions #1

Updated by Benoît PECCATTE almost 7 years ago

  • Related to Bug #12263: Wrong regex in FileTemplate technique added
Actions #2

Updated by Vincent MEMBRÉ over 6 years ago

  • Target version changed from 4.1.11 to 4.1.12
Actions #3

Updated by Vincent MEMBRÉ over 6 years ago

  • Target version changed from 4.1.12 to 4.1.13
Actions #4

Updated by Benoît PECCATTE over 6 years ago

  • Target version changed from 4.1.13 to 411
Actions #5

Updated by Benoît PECCATTE over 6 years ago

  • Target version changed from 411 to 4.1.13
Actions #6

Updated by Vincent MEMBRÉ over 6 years ago

  • Target version changed from 4.1.13 to 4.1.14
Actions #7

Updated by Benoît PECCATTE over 6 years ago

  • Target version changed from 4.1.14 to 4.1.15
Actions #8

Updated by François ARMAND about 6 years ago

  • Severity set to Minor - inconvenience | misleading | easy workaround
  • User visibility set to Infrequent - complex configurations | third party integrations
  • Effort required set to Very Small
  • Priority changed from 0 to 46

[problematic ex: <CONSTRAINT><REGEX> in filetemplate)

So, what we want is the regex to be always multiline.

We believe this is not a breaking change because for now, a regex with ^$ would never match multiline fields, so such an input can't have been added.

The change should be only to modify "case class RegexConstraint" to always add DOTALL and MULTILINE modifiers.

Actions #10

Updated by Vincent MEMBRÉ about 6 years ago

  • Target version changed from 4.1.15 to 4.1.16
Actions #11

Updated by Vincent MEMBRÉ about 6 years ago

  • Target version changed from 4.1.16 to 4.1.17
  • Priority changed from 46 to 45
Actions #12

Updated by Vincent MEMBRÉ about 6 years ago

  • Target version changed from 4.1.17 to 4.1.18
Actions #13

Updated by Vincent MEMBRÉ almost 6 years ago

  • Target version changed from 4.1.18 to 4.1.19
  • Priority changed from 45 to 44
Actions #14

Updated by Alexis Mousset almost 6 years ago

  • Target version changed from 4.1.19 to 4.1.20
  • Priority changed from 44 to 43
Actions #15

Updated by François ARMAND almost 6 years ago

  • Target version changed from 4.1.20 to 4.1.21
  • Priority changed from 43 to 42
Actions #16

Updated by Vincent MEMBRÉ over 5 years ago

  • Target version changed from 4.1.21 to 4.1.22
Actions #17

Updated by Vincent MEMBRÉ over 5 years ago

  • Target version changed from 4.1.22 to 4.1.23
Actions #18

Updated by Vincent MEMBRÉ over 5 years ago

  • Target version changed from 4.1.23 to 4.1.24
Actions #19

Updated by Vincent MEMBRÉ over 5 years ago

  • Target version changed from 4.1.24 to 588
Actions #20

Updated by Alexis Mousset over 5 years ago

  • Target version changed from 588 to 5.0.13
Actions #21

Updated by Vincent MEMBRÉ over 5 years ago

  • Target version changed from 5.0.13 to 5.0.14
Actions #22

Updated by Vincent MEMBRÉ about 5 years ago

  • Target version changed from 5.0.14 to 5.0.15
Actions #23

Updated by Vincent MEMBRÉ about 5 years ago

  • Target version changed from 5.0.15 to 5.0.16
Actions #24

Updated by Alexis Mousset almost 5 years ago

  • Target version changed from 5.0.16 to 5.0.17
Actions #25

Updated by Vincent MEMBRÉ over 4 years ago

  • Target version changed from 5.0.17 to 5.0.18
Actions #26

Updated by Vincent MEMBRÉ over 4 years ago

  • Target version changed from 5.0.18 to 5.0.19
Actions #27

Updated by Vincent MEMBRÉ over 4 years ago

  • Target version changed from 5.0.19 to 5.0.20
Actions #28

Updated by Vincent MEMBRÉ about 4 years ago

  • Target version changed from 5.0.20 to 797
Actions #29

Updated by Benoît PECCATTE over 3 years ago

  • Target version changed from 797 to 6.1.14
  • Priority changed from 42 to 43
Actions #30

Updated by Vincent MEMBRÉ over 3 years ago

  • Target version changed from 6.1.14 to 6.1.15
Actions #31

Updated by Vincent MEMBRÉ over 3 years ago

  • Target version changed from 6.1.15 to 6.1.16
  • Priority changed from 43 to 44
Actions #32

Updated by Vincent MEMBRÉ over 3 years ago

  • Target version changed from 6.1.16 to 6.1.17
Actions #33

Updated by Vincent MEMBRÉ about 3 years ago

  • Target version changed from 6.1.17 to 6.1.18
  • Priority changed from 44 to 45
Actions #34

Updated by Vincent MEMBRÉ about 3 years ago

  • Target version changed from 6.1.18 to 6.1.19
Actions #35

Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 6.1.19 to 6.1.20
  • Priority changed from 45 to 47
Actions #36

Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 6.1.20 to 6.1.21
Actions #37

Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 6.1.21 to old 6.1 issues to relocate
  • Priority changed from 47 to 48
Actions #38

Updated by Benoît PECCATTE about 1 year ago

  • Category changed from Techniques to Web - Config management
  • Target version changed from old 6.1 issues to relocate to 8.0.2
  • Priority changed from 48 to 0
  • Regression set to No
Actions #39

Updated by Vincent MEMBRÉ about 1 year ago

  • Target version changed from 8.0.2 to 8.0.3
Actions #40

Updated by Vincent MEMBRÉ about 1 year ago

  • Target version changed from 8.0.3 to 8.0.4
Actions #41

Updated by Vincent MEMBRÉ about 1 year ago

  • Target version changed from 8.0.4 to 8.0.5
Actions #42

Updated by Vincent MEMBRÉ 11 months ago

  • Target version changed from 8.0.5 to 8.0.6
Actions #43

Updated by Vincent MEMBRÉ 10 months ago

  • Target version changed from 8.0.6 to 8.0.7
Actions #44

Updated by Vincent MEMBRÉ 9 months ago

  • Target version changed from 8.0.7 to 8.0.8
Actions #45

Updated by Vincent MEMBRÉ 8 months ago

  • Target version changed from 8.0.8 to 8.0.9
Actions #46

Updated by Vincent MEMBRÉ 7 months ago

  • Target version changed from 8.0.9 to 8.0.10
Actions #47

Updated by Vincent MEMBRÉ 6 months ago

  • Target version changed from 8.0.10 to 1089
Actions #48

Updated by Alexis Mousset 5 months ago

  • Target version changed from 1089 to 8.1.6
Actions #49

Updated by Vincent MEMBRÉ 5 months ago

  • Target version changed from 8.1.6 to 8.1.7
Actions #50

Updated by Vincent MEMBRÉ 3 months ago

  • Target version changed from 8.1.7 to 8.1.8
Actions #51

Updated by Vincent MEMBRÉ about 1 month ago

  • Target version changed from 8.1.8 to 8.1.9
Actions #52

Updated by Vincent MEMBRÉ 25 days ago

  • Target version changed from 8.1.9 to 8.1.10
Actions #53

Updated by Vincent MEMBRÉ 20 days ago

  • Target version changed from 8.1.10 to 8.1.11
Actions

Also available in: Atom PDF