Skip to content

Commit

Permalink
Avoid 208 rule with unarchive module (#1089)
Browse files Browse the repository at this point in the history
As archives can also contain permissions (like tar ones), we should
avoid triggering 208 rule when unarchive module is used.

Fixes: #1064
  • Loading branch information
ssbarnea authored Nov 3, 2020
1 parent a3a4e52 commit 983549f
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion lib/ansiblelint/rules/MissingFilePermissionsRule.py
Original file line number Diff line number Diff line change
Expand Up @@ -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 = {
Expand Down

0 comments on commit 983549f

Please sign in to comment.