Project

General

Profile

Actions
Copy link

Bug #12857

closed

Generic method "File_remove" can delete empty directories

Added by Félix DALLIDET over 6 years ago. Updated over 2 years ago.

Status:
Released
Priority:
N/A
Assignee:
Nicolas CHARLES
Category:
Generic methods
Target version:
4.1.17
Pull Request:
https://github.com/Normation/ncf/pull...
Severity:
UX impact:
User visibility:
Effort required:
Very Small
Priority:
0
Name check:
Fix check:
Regression:

Description

if we build a dir like this:

mkdir /tmp/toRemove

Apply a directive based from the GM "file_absent" on the file "/tmp/toRemove", Rudder will remove the folder.
Now, if we make the same directory and add a file in it:

mkdir /tmp/toRemove
touch /tmp/toRemove/afile

Rudder does not remove the directory but returns a compliant report.

This is most likely unwanted.

Actions
Copy link
 #1

Updated by Alexis Mousset about 6 years ago

  • Effort required set to Very Small

We can add a simple test before actual action.

Actions
Copy link
 #2

Updated by Alexis Mousset about 6 years ago

  • Target version set to 4.1.16
Actions
Copy link
 #3

Updated by Vincent MEMBRÉ about 6 years ago

  • Target version changed from 4.1.16 to 4.1.17
Actions
Copy link
 #4

Updated by Nicolas CHARLES about 6 years ago

  • Status changed from New to In progress
  • Assignee set to Nicolas CHARLES
Actions
Copy link
 #5

Updated by Nicolas CHARLES about 6 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/874
Actions
Copy link
 #6

Updated by Rudder Quality Assistant about 6 years ago

  • Assignee changed from Alexis Mousset to Nicolas CHARLES
Actions
Copy link
 #7

Updated by Nicolas CHARLES about 6 years ago

  • Status changed from Pending technical review to Pending release

Applied in changeset commit:d2556b48bf1f94828a624549bc684147abacfb6c.

Actions
Copy link
 #8

Updated by Vincent MEMBRÉ about 6 years ago

  • Translation missing: en.field_tag_list set to Next minor release
Actions
Copy link
 #9

Updated by François ARMAND almost 6 years ago

  • Subject changed from File_absent GM can remove empty directories to Generic method "File_remove" can delete empty directories
Actions
Copy link
 #10

Updated by Vincent MEMBRÉ almost 6 years ago

  • Status changed from Pending release to Released
This bug has been fixed in Rudder 4.1.17, 4.3.7 and 5.0.3 which were released today. 
Changelog
 Changelog
 Changelog
Actions
Copy link
 #11

Updated by Alexis Mousset over 2 years ago

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

Also available in: Atom PDF