Project

General

Profile

Actions
Copy link

Bug #12269

open

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

Added by Benoît PECCATTE over 5 years ago. Updated about 1 year ago.

Status:
New
Priority:
N/A
Assignee:
-
Category:
Techniques
Target version:
old 6.1 issues to relocate
Pull Request:
Severity:
Minor - inconvenience | misleading | easy workaround
UX impact:
User visibility:
Infrequent - complex configurations | third party integrations
Effort required:
Very Small
Priority:
48
Regression:

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 open — 1 closed)

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

Updated by Benoît PECCATTE over 5 years ago

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

Updated by Vincent MEMBRÉ over 5 years ago

  • Target version changed from 4.1.11 to 4.1.12
Actions
Copy link
 #3

Updated by Vincent MEMBRÉ over 5 years ago

  • Target version changed from 4.1.12 to 4.1.13
Actions
Copy link
 #4

Updated by Benoît PECCATTE over 5 years ago

  • Target version changed from 4.1.13 to 411
Actions
Copy link
 #5

Updated by Benoît PECCATTE over 5 years ago

  • Target version changed from 411 to 4.1.13
Actions
Copy link
 #6

Updated by Vincent MEMBRÉ about 5 years ago

  • Target version changed from 4.1.13 to 4.1.14
Actions
Copy link
 #7

Updated by Benoît PECCATTE about 5 years ago

  • Target version changed from 4.1.14 to 4.1.15
Actions
Copy link
 #8

Updated by François ARMAND almost 5 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
Copy link
 #10

Updated by Vincent MEMBRÉ almost 5 years ago

  • Target version changed from 4.1.15 to 4.1.16
Actions
Copy link
 #11

Updated by Vincent MEMBRÉ almost 5 years ago

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

Updated by Vincent MEMBRÉ almost 5 years ago

  • Target version changed from 4.1.17 to 4.1.18
Actions
Copy link
 #13

Updated by Vincent MEMBRÉ over 4 years ago

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

Updated by Alexis Mousset over 4 years ago

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

Updated by François ARMAND over 4 years ago

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

Updated by Vincent MEMBRÉ over 4 years ago

  • Target version changed from 4.1.21 to 4.1.22
Actions
Copy link
 #17

Updated by Vincent MEMBRÉ over 4 years ago

  • Target version changed from 4.1.22 to 4.1.23
Actions
Copy link
 #18

Updated by Vincent MEMBRÉ over 4 years ago

  • Target version changed from 4.1.23 to 4.1.24
Actions
Copy link
 #19

Updated by Vincent MEMBRÉ about 4 years ago

  • Target version changed from 4.1.24 to 588
Actions
Copy link
 #20

Updated by Alexis Mousset about 4 years ago

  • Target version changed from 588 to 5.0.13
Actions
Copy link
 #21

Updated by Vincent MEMBRÉ about 4 years ago

  • Target version changed from 5.0.13 to 5.0.14
Actions
Copy link
 #22

Updated by Vincent MEMBRÉ almost 4 years ago

  • Target version changed from 5.0.14 to 5.0.15
Actions
Copy link
 #23

Updated by Vincent MEMBRÉ almost 4 years ago

  • Target version changed from 5.0.15 to 5.0.16
Actions
Copy link
 #24

Updated by Alexis Mousset over 3 years ago

  • Target version changed from 5.0.16 to 5.0.17
Actions
Copy link
 #25

Updated by Vincent MEMBRÉ over 3 years ago

  • Target version changed from 5.0.17 to 5.0.18
Actions
Copy link
 #26

Updated by Vincent MEMBRÉ about 3 years ago

  • Target version changed from 5.0.18 to 5.0.19
Actions
Copy link
 #27

Updated by Vincent MEMBRÉ about 3 years ago

  • Target version changed from 5.0.19 to 5.0.20
Actions
Copy link
 #28

Updated by Vincent MEMBRÉ almost 3 years ago

  • Target version changed from 5.0.20 to 797
Actions
Copy link
 #29

Updated by Benoît PECCATTE about 2 years ago

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

Updated by Vincent MEMBRÉ about 2 years ago

  • Target version changed from 6.1.14 to 6.1.15
Actions
Copy link
 #31

Updated by Vincent MEMBRÉ about 2 years ago

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

Updated by Vincent MEMBRÉ about 2 years ago

  • Target version changed from 6.1.16 to 6.1.17
Actions
Copy link
 #33

Updated by Vincent MEMBRÉ almost 2 years ago

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

Updated by Vincent MEMBRÉ almost 2 years ago

  • Target version changed from 6.1.18 to 6.1.19
Actions
Copy link
 #35

Updated by Vincent MEMBRÉ over 1 year ago

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

Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 6.1.20 to 6.1.21
Actions
Copy link
 #37

Updated by Vincent MEMBRÉ about 1 year ago

  • Target version changed from 6.1.21 to old 6.1 issues to relocate
  • Priority changed from 47 to 48
Actions
Copy link

Also available in: Atom PDF