Skip to content

Conversation

@edops973
Copy link

@edops973 edops973 commented Mar 9, 2025

Card

BishopFox#85 : Associate container to ECS task in env-vars or ecs-task definitions

Details

  • Added container name column to ecs-tasks command
    Unittests for ecs-tasks command still work
image
  • Added task definition column to env-vars command
    env-vars example output
image

@sethsec
Copy link
Owner

sethsec commented Mar 12, 2025

Hey @edops973 - For the env-vars command it feels a little too heavy handed to make a new column for task definition when that only applies for ecs tasks. What if we instead merge that in the name. Like instead of name = container.name, maybe name = container.name/task-definition.

Name
nginx-container/nginx-task:4

The ecs-tasks stuff looks amazing as is!

@edops973
Copy link
Author

hi @sethsec , its a great suggestion. I'll update it

@sethsec
Copy link
Owner

sethsec commented Mar 12, 2025

Works great. Thank you @edops973 !

@sethsec sethsec merged commit b139513 into sethsec:main Mar 12, 2025
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