Skip to content

Commit

Permalink
[Fix] - forget to assigin args to aws_ec2_instance_id for function aw…
Browse files Browse the repository at this point in the history
…s_ssm_connection_ec2 and add function to remove auto load current assume role credentials
  • Loading branch information
lamhaison committed Feb 27, 2023
1 parent 69c4660 commit aab3fee
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 0 deletions.
4 changes: 4 additions & 0 deletions common/common.sh
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,10 @@ aws_assume_role_disable_fast_mode() {
export aws_assume_role_print_account_info=true
}

aws_assume_role_disable_load_current_assume_role_for_new_tab() {
rm -rf ${aws_cli_current_assume_role_name} >/dev/null
}

aws_assume_role_disable_show_detail_commandline() {
export aws_show_commandline=false
}
Expand Down
1 change: 1 addition & 0 deletions services/ssm.sh
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@ aws_ssm_list_parameters() {
}

aws_ssm_connection_ec2() {
aws_ec2_instance_id=$1
aws_commandline_logging "\
aws ssm start-session --target ${aws_ec2_instance_id:?'aws_ec2_instance_id is unset or empty'}
"
Expand Down

0 comments on commit aab3fee

Please sign in to comment.