duo-labs / cloudmapper

CloudMapper helps you analyze your Amazon Web Services (AWS) environments.
BSD 3-Clause "New" or "Revised" License
6k stars 810 forks source link

Counting DynamoDB Tables #766

Open ps9786 opened 4 years ago

ps9786 commented 4 years ago

Hi, I have searched the code and the yaml's but cant see any way to count the number of DynamoDB tables. Do I need to add a custom scipt?

Apologies if this has been asked before I did search for DynamoDB in the issues.

0xdabbad00 commented 4 years ago

CloudMapper does not currently collect info about DynamoDB tables, as there hasn't been anything I know to audit about them and haven't had another use for them within CloudMapper. You would need to add lines to the collect configuration: https://github.com/duo-labs/cloudmapper/blob/main/collect_commands.yaml

And then add lines to the stats configuration: https://github.com/duo-labs/cloudmapper/blob/main/stats_config.yaml

If you want to send a PR for this I'll get it merged.

ps9786 commented 4 years ago

Thanks for the reply and info.

I successfully managed to get a count of tables but when trying to get the "Arn" for the table in order to get the Tags, I am failing.

I have this in collect

- Service: dynamodb
  Request: list-tables
- Service: dynamodb
  Request: describe-table
  Parameters:
  - Name: TableName
    Value: dynamodb-list-tables.json |.TableNames[]?|.

I can see the table names when using jq, what am I doing wrong?

Thank you.

Paul

0xdabbad00 commented 4 years ago

This looks right, maybe just the space after the .json needs to be removed? For testing, you can comment out or delete everything else in that collect_commands.yaml file, and include the --regions flag to limit the regions collected from, which will make things go faster. Then you can add some print statements around here: https://github.com/duo-labs/cloudmapper/blob/9e91a9a7780d635478106a4ec46c1fc92e222be2/commands/collect.py#L456

That should help you debug.

ps9786 commented 4 years ago

Thanks, I'll give that ago.

Paul

w0rmr1d3r commented 3 years ago

Did this ever get solved? @ps9786 @0xdabbad00 Shall we create the PR for it?

ps9786 commented 3 years ago

Ramon,

Yes, I just added -

Service: dynamodb Request: list-tables

to the collect yaml.

Thanks,

Paul

On Mon, 15 Mar 2021 at 14:41, Ramon @.***> wrote:

Did this ever get solved? @ps9786 https://github.com/ps9786 @0xdabbad00 https://github.com/0xdabbad00 Shall we create the PR for it?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/duo-labs/cloudmapper/issues/766#issuecomment-799474395, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABYTOFSYDDPAXRLXX2X253TTDYMCXANCNFSM4RUUGXIQ .

w0rmr1d3r commented 3 years ago

Sorry @ps9786 but I can't find it, did you add it to this repo or to your fork?

ps9786 commented 3 years ago

Ramon,

Ah you got me there. I did it locally (bad I know) but did not think it was a resource that others would be interested in.

Paul

On Mon, 15 Mar 2021 at 15:50, Ramon @.***> wrote:

Sorry @ps9786 https://github.com/ps9786 but I can't find it, did you add it to this repo or to your fork?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/duo-labs/cloudmapper/issues/766#issuecomment-799528142, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABYTOFRKXEQTESD7CKWOG6DTDYUDFANCNFSM4RUUGXIQ .

w0rmr1d3r commented 3 years ago

Alright @ps9786 ! I created the PR https://github.com/duo-labs/cloudmapper/pull/858 adding what you said in this issue so this can get solved!

ps9786 commented 3 years ago

Ramon,

Thanks, is there anything I need to do?

Paul

On Mon, 15 Mar 2021 at 16:21, Ramon @.***> wrote:

Alright @ps9786 https://github.com/ps9786 ! I created the PR #858 https://github.com/duo-labs/cloudmapper/pull/858 adding what you said in this issue so this can get solved!

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/duo-labs/cloudmapper/issues/766#issuecomment-799552065, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABYTOFUZGBQMRNTVJKBUXODTDYXXLANCNFSM4RUUGXIQ .

w0rmr1d3r commented 3 years ago

Hello Paul! mmm.... review the PR and approve it if it's what you did, so the rest of us using this awesome tool can benefit from those changes too!

Thank you! :)

0xdabbad00 commented 3 years ago

I merged the PR for collecting the tables, but it doesn't count the tables, so I'll leave this open.