Avoid 208 rule with unarchive module (#1089)

ssbarnea · web-flow · commit 983549ff38c4 · 2020-11-03T13:52:29.000+01:00
As archives can also contain permissions (like tar ones), we should avoid triggering 208 rule when unarchive module is used. Fixes: #1064
diff --git a/lib/ansiblelint/rules/MissingFilePermissionsRule.py b/lib/ansiblelint/rules/MissingFilePermissionsRule.py
@@ -48,7 +48,7 @@ class MissingFilePermissionsRule(AnsibleLintRule):
         'file',
         'replace',  # implicit preserve behavior but mode: preserve is invalid
         'template',  # supports preserve
-        'unarchive',
+        # 'unarchive',  # disabled because .tar.gz files can have permissions inside
     }
 
     _modules_with_create = {

Original file line number	Diff line number	Diff line change
`@@ -48,7 +48,7 @@ class MissingFilePermissionsRule(AnsibleLintRule):`
`48`	`48`	`'file',`
`49`	`49`	`'replace', # implicit preserve behavior but mode: preserve is invalid`
`50`	`50`	`'template', # supports preserve`
`51`		`- 'unarchive',`
	`51`	`+ # 'unarchive', # disabled because .tar.gz files can have permissions inside`
`52`	`52`	`}`
`53`	`53`
`54`	`54`	`_modules_with_create = {`