Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

More no_log changes #471

Merged
merged 2 commits into from
Mar 12, 2021
Merged

Conversation

tremble
Copy link
Contributor

@tremble tremble commented Mar 12, 2021

SUMMARY
  • aws_secret's secret parameter contains sensitive data and shouldn't be logged.
  • s3_sync's key_prefix matches 'key' so triggers a false positive with the latest sanity tests
ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

s3_sync
aws_secret

ADDITIONAL INFORMATION

Thanks to @felixfontein for the heads up
ansible/ansible#73508

@ansibullbot
Copy link

@ansibullbot ansibullbot added bug This issue/PR relates to a bug community_review module module needs_triage owner_pr PR created by owner/maintainer plugins plugin (any type) small_patch Hopefully easy to review labels Mar 12, 2021
@tremble tremble merged commit f7a818b into ansible-collections:main Mar 12, 2021
danquixote pushed a commit to danquixote/community.aws that referenced this pull request May 16, 2021
* More no_log changes
* changelog
@tremble tremble deleted the no_log/2021-03-12 branch June 29, 2021 12:56
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request Jul 19, 2021
* More no_log changes
* changelog
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request Jul 19, 2021
* More no_log changes
* changelog
danielcotton pushed a commit to danielcotton/community.aws that referenced this pull request Nov 23, 2021
* More no_log changes
* changelog
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue/PR relates to a bug community_review module module needs_triage owner_pr PR created by owner/maintainer plugins plugin (any type) small_patch Hopefully easy to review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants