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

Resource updates #971

Draft
wants to merge 16 commits into
base: main
Choose a base branch
from
Draft

Resource updates #971

wants to merge 16 commits into from

Conversation

Amndeep7
Copy link

@Amndeep7 Amndeep7 commented Jun 23, 2023

Description

  • Adds a resource for the IAM Credential Report endpoint
  • Adds a resource for the Account endpoint
  • Fix docs for IAM Policy
  • Fix docs/example for IAM Users (it's currently the one from IAM User)
  • Add docs for Account
  • Add docs for IAM Credential Report
  • Fix the resource_id and to_s functions for cloud watch log metric filter so that it handles the case when there are no metric filters
  • Fix iam_access_keys

Please describe what this change achieves. Ensure you have read the Contributing to InSpec AWS document before submitting.

Issues Resolved

List any existing issues this PR resolves, or any Discourse or StackOverflow discussion that's relevant

Check List

Please fill box or appropriate ([x]) or mark N/A.

Signed-off-by: Amndeep Singh Mann <amann@mitre.org>
Signed-off-by: Amndeep Singh Mann <amann@mitre.org>
@netlify
Copy link

netlify bot commented Jun 23, 2023

Deploy Preview for inspec-aws canceled.

Name Link
🔨 Latest commit e4b35ba
🔍 Latest deploy log https://app.netlify.com/sites/inspec-aws/deploys/65537d2812e1d3000834411c

Signed-off-by: Amndeep Singh Mann <amann@mitre.org>
Signed-off-by: Amndeep Singh Mann <amann@mitre.org>
@@ -10,6 +10,9 @@ gem 'bundle'
# Note if the gem required is commented out, please raise a PR against the train-aws repo first
# In the mean time the gem can be added here for local development

# TODO: remove when https://github.com/inspec/train-aws/pull/457 is merged
gem 'aws-sdk-account', '~> 1.14'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

inspec/train-aws#457 is merged.Please update Gemfile.

Amndeep7 and others added 12 commits June 29, 2023 02:20
Signed-off-by: Amndeep Singh Mann <amann@mitre.org>
Signed-off-by: Amndeep Singh Mann <amann@mitre.org>
…variable + made the ever/never funcs use ||=

Signed-off-by: Amndeep Singh Mann <amann@mitre.org>
Signed-off-by: Amndeep Singh Mann <amann@mitre.org>
Signed-off-by: Aaron Lippold <lippold@gmail.com>
Signed-off-by: Aaron Lippold <lippold@gmail.com>
* updated and fixed the aws_accounts.rb resources to use standard styles
  and filterTable
* Linted with 'cookstyle -A libraries/*.rb'
* renamed the 'aws_account' resource to:
  - aws_primary_contact
  - aws_billing_contact
  - aws_operations_contact
  - aws_security_contact

Signed-off-by: Aaron Lippold <lippold@gmail.com>
…ot configured

Signed-off-by: Aaron Lippold <lippold@gmail.com>
Signed-off-by: Aaron Lippold <lippold@gmail.com>
* removed unneeded if clause in init method given fix

Signed-off-by: Aaron Lippold <lippold@gmail.com>
Signed-off-by: Aaron Lippold <lippold@gmail.com>
@aaronlippold
Copy link

This PR can be closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants