Fix issue 527 - Githubissues

duo-labs / cloudmapper

CloudMapper helps you analyze your Amazon Web Services (AWS) environments.

BSD 3-Clause "New" or "Revised" License

5.9k stars 800 forks source link

Fix issue 527 #852

Closed w0rmr1d3r closed 2 years ago

w0rmr1d3r commented 3 years ago

What?

Fixes issue https://github.com/duo-labs/cloudmapper/issues/527

Other

Optimized imports
Removed unused parameters from some functions
Added some docstrings
Reduced conditional redundancy by merging conditionals

0xdabbad00 commented 2 years ago

This PR unfortunately has too many changes for me to be able to review it.

w0rmr1d3r commented 2 years ago

Hello @0xdabbad00 , do you want me to split it into smaller PRs?

Thank you!

0xdabbad00 commented 2 years ago

CloudMapper is mostly on life-support at this point, so I'm mostly trying to only merge in bug fixes or simple needed features. This is a nice to have, but not needed.