Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactoring/aws/cloudtrail #219

Merged
merged 12 commits into from
Mar 4, 2019

Conversation

Aboisier
Copy link
Contributor

@Aboisier Aboisier commented Mar 4, 2019

This PR migrates CloudTrail to the new architecture, see #183 .

@Aboisier Aboisier changed the title Refactoring/aws/ Refactoring/aws/cloudtrail Mar 4, 2019
@Aboisier Aboisier self-assigned this Mar 4, 2019
@Aboisier Aboisier added this to the Iteration #4 milestone Mar 4, 2019
@codecov-io
Copy link

codecov-io commented Mar 4, 2019

Codecov Report

Merging #219 into refactoring/resource-configs will increase coverage by 0.12%.
The diff coverage is 30.43%.

Impacted file tree graph

@@                       Coverage Diff                       @@
##           refactoring/resource-configs    #219      +/-   ##
===============================================================
+ Coverage                         28.27%   28.4%   +0.12%     
===============================================================
  Files                                88      90       +2     
  Lines                              5566    5570       +4     
===============================================================
+ Hits                               1574    1582       +8     
+ Misses                             3992    3988       -4
Impacted Files Coverage Δ
ScoutSuite/providers/aws/provider.py 10.27% <0%> (-0.03%) ⬇️
ScoutSuite/providers/aws/facade/facade.py 57.89% <100%> (+4.95%) ⬆️
ScoutSuite/providers/aws/configs/services.py 49.18% <25%> (ø) ⬆️
...uite/providers/aws/resources/cloudtrail/service.py 27.65% <27.65%> (ø)
ScoutSuite/providers/aws/facade/cloudtrail.py 33.33% <33.33%> (ø)
ScoutSuite/providers/base/utils.py 40% <40%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 70108b4...123b5b4. Read the comment docs.

@Aboisier Aboisier changed the base branch from refactoring/aws/ec2 to refactoring/resource-configs March 4, 2019 16:46
from ScoutSuite.providers.aws.resources.regions import Regions
from ScoutSuite.providers.aws.resources.resources import AWSResources
from ScoutSuite.providers.aws.facade.facade import AWSFacade
from ScoutSuite.providers.base.utils import get_non_provider_id
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
from ScoutSuite.providers.base.utils import get_non_provider_id
from ScoutSuite.providers.utils import get_non_provider_id

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(There's already a utils.py file defining get_non_provider_id there.)

Copy link
Contributor

@Remi05 Remi05 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work! 👌

@Aboisier Aboisier requested a review from misg March 4, 2019 21:34
Copy link
Contributor

@vifor2 vifor2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Neato 👌

@Aboisier Aboisier merged commit a13ecd1 into refactoring/resource-configs Mar 4, 2019
@Aboisier Aboisier deleted the refactoring/aws/cloudtrail branch March 4, 2019 23:05
@Aboisier Aboisier mentioned this pull request Mar 13, 2019
53 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants