8000 Release v4.1.0 by Aboisier · Pull Request #142 · nccgroup/ScoutSuite · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Release v4.1.0 #142

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

Merged
merged 185 commits into from
Feb 5, 2019
Merged

Release v4.1.0 #142

merged 185 commits into from
Feb 5, 2019

Conversation

Aboisier
Copy link
Contributor
@Aboisier Aboisier commented Feb 4, 2019

Release v4.1.0.

Please also see the proprietary PR.

x4v13r64 and others added 30 commits January 8, 2019 18:11
…ect-html

Incorrect HTML for VPC Network ACLs
The loop was iterating over the instances ids instead of over the resources
…nterfaces, closes #10

Bugfix/10 missing ec2 network interfaces
@Aboisier Aboisier added this to the Iteration #1 milestone Feb 4, 2019
@Aboisier Aboisier self-assigned this Feb 4, 2019
@Aboisier Aboisier changed the title Release/v4.1.0 Release v4.1.0 Feb 4, 2019
@x4v13r64
Copy link
Collaborator
x4v13r64 commented Feb 4, 2019

I would wait until the CLI has been refactored (i.e. until https://github.com/nccgroup/ScoutSuite/tree/refactor/127-cli-refactoring is merged) before making the 4.1.0 release.

Main reason is to that Azure authentication doesn't change too much from 4.1.0 to 4.2.0. Sounds good?

@codecov-io
Copy link
codecov-io commented Feb 4, 2019

Codecov Report

Merging #142 into master will decrease coverage by 0.1%.
The diff coverage is 33.21%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #142      +/-   ##
==========================================
- Coverage   30.46%   30.36%   -0.11%     
==========================================
  Files          63       64       +1     
  Lines        4034     4206     +172     
==========================================
+ Hits         1229     1277      +48     
- Misses       2805     2929     +124
Impacted Files Coverage Δ
ScoutSuite/output/console.py 13.41% <ø> (+0.31%) ⬆️
ScoutSuite/providers/aws/services/cloudtrail.py 18.33% <0%> (-0.97%) ⬇️
ScoutSuite/providers/gcp/services/cloudstorage.py 16% <0%> (+0.9%) ⬆️
ScoutSuite/__main__.py 66.32% <0%> (ø) ⬆️
ScoutSuite/providers/gcp/services/computeengine.py 11.42% <0%> (ø) ⬆️
ScoutSuite/providers/aws/configs/base.py 52.63% <0%> (+5.01%) ⬆️
ScoutSuite/providers/aws/configs/services.py 54.16% <0%> (+2.16%) ⬆️
ScoutSuite/providers/__init__.py 50% <0%> (-10%) ⬇️
ScoutSuite/providers/azure/configs/base.py 50% <0%> (+4.54%) ⬆️
ScoutSuite/core/processingengine.py 80.39% <100%> (+0.8%) ⬆️
... and 25 more

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 1d6df9b...5b72e1b. Read the comment docs.

@misg
Copy link
Contributor
misg commented Feb 4, 2019

Tested it with Azure and GCP and everything looks fine. 💯
Could someone test it with AWS?

@x4v13r64
Copy link
Collaborator
x4v13r64 commented Feb 4, 2019

Could someone test it with AWS?

Looks fine to me 🥇

@Aboisier Aboisier merged commit 476ab3f into master Feb 5, 2019
@Aboisier Aboisier deleted the release/v4.1.0 branch February 5, 2019 15:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants
0