Project

General

Profile

Actions

Bug #3759

closed

Change Request could display error in the logs if its directive is based on Techniques using fixed regex (IP, Mail)

Added by Vincent MEMBRÉ over 9 years ago. Updated almost 8 years ago.

Status:
Released
Priority:
1
Category:
Web - Config management
Target version:
Severity:
UX impact:
User visibility:
Effort required:
Priority:
Regression:

Description

When making a change request we have to serialize the directive and the technique on which this directive is based.

But some Technique fields (fixed regexp : mail and IP) are not correctly parsed

the problem is that we serialize their regexp but the parser does not accept the 'fixed regexp' with a regexp field (think it's a redefinition of a fixed regexp but by definition fixed regexp are ... fixed )

Two ways to solve this : ignore regexp field for fixed regexp in parser or don't serialize 'fixed regexp' with their regexp so the parser won't have problems.

Actions #1

Updated by Vincent MEMBRÉ over 9 years ago

  • Status changed from New to Pending technical review
  • Assignee changed from Vincent MEMBRÉ to François ARMAND
Actions #2

Updated by Vincent MEMBRÉ over 9 years ago

  • Status changed from Pending technical review to Pending release
  • % Done changed from 0 to 100

Applied in changeset clerk:commit:78bc1293154d7a5ad70f5bc3c30bc198429a2b48.

Actions #3

Updated by Anonymous over 9 years ago

Applied in changeset clerk:commit:27195086b4961af1faf026fc365abf29c9e03c9b.

Actions #4

Updated by Nicolas PERRON over 9 years ago

  • Subject changed from Directives based on Techniques using Fixed regex (IP, Mail) are not correctly seriallized into change request to Change Request could display error in the log if its directive is based on Techniques using fixed regex (IP, Mail)
Actions #5

Updated by Nicolas PERRON over 9 years ago

  • Subject changed from Change Request could display error in the log if its directive is based on Techniques using fixed regex (IP, Mail) to Change Request could display error in the logs if its directive is based on Techniques using fixed regex (IP, Mail)
Actions #6

Updated by Nicolas PERRON over 9 years ago

  • Status changed from Pending release to Released
Actions #7

Updated by Nicolas PERRON over 9 years ago

This bug has been fixed in Rudder 2.6.3, which was released today.
Check out:

Actions #8

Updated by Benoît PECCATTE almost 8 years ago

  • Category changed from 14 to Web - Config management
Actions

Also available in: Atom PDF