Project

General

Profile

Actions

Bug #18451

closed

techniques in audit don't report correctly for editing values in files

Added by Nicolas CHARLES about 4 years ago. Updated over 2 years ago.

Status:
Released
Priority:
N/A
Category:
Generic methods
Target version:
Severity:
UX impact:
User visibility:
Effort required:
Priority:
0
Name check:
To do
Fix check:
To do
Regression:

Description

attached file show the issue
when body action field_edits_action is uncommented, the class success is set, rather than error

we should probably remove the edit_lines audit part, with careful testing


Files

promises.cf (3.09 KB) promises.cf Nicolas CHARLES, 2020-10-30 12:04

Related issues 1 (0 open1 closed)

Related to Rudder - Bug #16181: Missing edit_lines promises in dry-run libReleasedNicolas CHARLESActions
Actions #1

Updated by Nicolas CHARLES about 4 years ago

  • Project changed from Rudder to 41
  • Category changed from System techniques to Generic methods
Actions #2

Updated by Nicolas CHARLES about 4 years ago

  • Status changed from New to In progress
  • Assignee set to Nicolas CHARLES
Actions #3

Updated by Nicolas CHARLES about 4 years ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from Nicolas CHARLES to Alexis Mousset
  • Pull Request set to https://github.com/Normation/ncf/pull/1257
Actions #4

Updated by Nicolas CHARLES about 4 years ago

  • Status changed from Pending technical review to Pending release

Applied in changeset commit:62ece0222fd8f4ef9a3ee1608fe7988bb109cacd.

Actions #5

Updated by Nicolas CHARLES about 4 years ago

  • Related to Bug #16181: Missing edit_lines promises in dry-run lib added
Actions #6

Updated by Vincent MEMBRÉ almost 4 years ago

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 6.1.7 which was released today.

Actions #7

Updated by Alexis Mousset over 2 years ago

  • Project changed from 41 to Rudder
  • Category changed from Generic methods to Generic methods
Actions

Also available in: Atom PDF