Skip to content

Conversation

lamhaison
Copy link
Owner

This pull request standardizes the function definitions across multiple shell scripts by explicitly using the function keyword. Additionally, it introduces new utility functions for AWS IAM, logging, and other AWS services, enhancing functionality and user interaction. Below is a categorized summary of the most important changes:

Standardization of Function Definitions

  • Updated all function definitions in common/common.sh, common/logging.sh, common/peco.sh, services/assume_role.sh, services/iam.sh, and services/logs.sh to explicitly use the function keyword for consistency. [1] [2] [3] [4] [5] [6]

Enhancements to AWS IAM Functionality

  • Added new functions to retrieve and list IAM user policies, including both managed and inline policies, and to display group policies for users. These include aws_iam_user_list_policies, aws_iam_user_list_policies_with_hint, and aws_iam_user_list_policies_all.
  • Introduced a new function aws_iam_user_list_access_keys_with_hint for listing IAM user access keys interactively using peco.

Improvements to AWS Logging

  • Added aws_logs_rm_group_instruction and aws_logs_rm_group_instruction_with_hint to generate and interactively select commands for deleting AWS CloudWatch log groups.

New Interactive Peco Menus for AWS Services

  • Added peco_aws_iam_user_list, peco_aws_iam_group_list, peco_aws_iam_policy_list, and peco_aws_iam_instance_profile_list for interactively selecting IAM entities.
  • Enhanced existing peco functions to support additional AWS services, including CloudFormation, DynamoDB, and SQS.

Additional Utility Functions

  • Introduced aws_assume_role_set_name_with_hint in services/assume_role.sh for setting assume role names interactively.

These changes improve code consistency, enhance user interaction via peco, and expand the functionality of AWS-related scripts.

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.

2 participants