-
Notifications
You must be signed in to change notification settings - Fork 17
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
Release/v1.3.0 (#39) #40
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* PLA-17940 updating constraints and fixing s3_enable_access_logging to… (#3) * PLA-17940 updating constraints and fixing s3_enable_access_logging to not remediate the logging target bucket * PLA-17940 fixing copyright info * Updated readme to have link for reporting issues (#4) Co-authored-by: svikramjeet <[email protected]> * S3 access logs permissions (#6) * Update minimum permissions for the job * Add more logs when permission is missing Co-authored-by: Mohammad Zuber Khan <[email protected]> * change the way cloudAccountId is parsed from Job Paramaters (#9) Co-authored-by: Mohammad Zuber Khan <[email protected]> * PLA-16779: Add remediation job for azure security group port 22 (#10) * PLA-16779: Add remediation job for azure security group port 22 * add test dependencies * update README.md * add deployment info to the README * add rule information Co-authored-by: Mohammad Zuber Khan <[email protected]> * Add remediation job for closing port 22 for VM (#11) * PLA-18743: Add remediation job for closing port 22 for VM * add tests for azure_vm_close_port_22 to tox * add deployment details Co-authored-by: Mohammad Zuber Khan <[email protected]> * Add remediation jobs for storage and RDP violations (#12) * add remediation job for closing RDP access * Add remediation job for remove public access for blob * Add remediation job to allow only https traffic to storage account * update READMEs to fix broken links Co-authored-by: Mohammad Zuber Khan <[email protected]> * fix the parameters passed for remediation (#13) Co-authored-by: Mohammad Zuber Khan <[email protected]> * add check for existing permissions before adding new (#15) Co-authored-by: Mohammad Zuber Khan <[email protected]> * Fix ports range for network security groups (#19) * handle the case when the security rule port is a range * Add minimum permissions for each remediation jobs * add link to built in roles Co-authored-by: Mohammad Zuber Khan <[email protected]> * update the remediation job payload (#21) Co-authored-by: Mohammad Zuber Khan <[email protected]> * PLA-20459: Add rule information for the remediation job (#22) * update the remediation job payload (#21) Co-authored-by: Mohammad Zuber Khan <[email protected]> * PLA-20459: Add rule information for the remediation job * add another rule remediated by the job Co-authored-by: Mohammad Zuber Khan <[email protected]> * Master -> Dev Merge (#23) * Release/v1.0.0 (#7) * PLA-17940 updating constraints and fixing s3_enable_access_logging to… (#3) * PLA-17940 updating constraints and fixing s3_enable_access_logging to not remediate the logging target bucket * PLA-17940 fixing copyright info * Updated readme to have link for reporting issues (#4) Co-authored-by: svikramjeet <[email protected]> * S3 access logs permissions (#6) * Update minimum permissions for the job * Add more logs when permission is missing Co-authored-by: Mohammad Zuber Khan <[email protected]> Co-authored-by: Paul Allen <[email protected]> Co-authored-by: svikramjeet <[email protected]> Co-authored-by: Zuber <[email protected]> Co-authored-by: Mohammad Zuber Khan <[email protected]> * Release/v1.1.0 (#17) * PLA-17940 updating constraints and fixing s3_enable_access_logging to… (#3) * PLA-17940 updating constraints and fixing s3_enable_access_logging to not remediate the logging target bucket * PLA-17940 fixing copyright info * Updated readme to have link for reporting issues (#4) Co-authored-by: svikramjeet <[email protected]> * S3 access logs permissions (#6) * Update minimum permissions for the job * Add more logs when permission is missing Co-authored-by: Mohammad Zuber Khan <[email protected]> * change the way cloudAccountId is parsed from Job Paramaters (#9) Co-authored-by: Mohammad Zuber Khan <[email protected]> * PLA-16779: Add remediation job for azure security group port 22 (#10) * PLA-16779: Add remediation job for azure security group port 22 * add test dependencies * update README.md * add deployment info to the README * add rule information Co-authored-by: Mohammad Zuber Khan <[email protected]> * Add remediation job for closing port 22 for VM (#11) * PLA-18743: Add remediation job for closing port 22 for VM * add tests for azure_vm_close_port_22 to tox * add deployment details Co-authored-by: Mohammad Zuber Khan <[email protected]> * Add remediation jobs for storage and RDP violations (#12) * add remediation job for closing RDP access * Add remediation job for remove public access for blob * Add remediation job to allow only https traffic to storage account * update READMEs to fix broken links Co-authored-by: Mohammad Zuber Khan <[email protected]> * fix the parameters passed for remediation (#13) Co-authored-by: Mohammad Zuber Khan <[email protected]> * add check for existing permissions before adding new (#15) Co-authored-by: Mohammad Zuber Khan <[email protected]> * Fix ports range for network security groups (#19) (#20) * handle the case when the security rule port is a range * Add minimum permissions for each remediation jobs * add link to built in roles Co-authored-by: Mohammad Zuber Khan <[email protected]> Co-authored-by: Zuber <[email protected]> Co-authored-by: Mohammad Zuber Khan <[email protected]> Co-authored-by: Paul Allen <[email protected]> Co-authored-by: svikramjeet <[email protected]> Co-authored-by: Zuber <[email protected]> Co-authored-by: Mohammad Zuber Khan <[email protected]> Co-authored-by: Paul Allen <[email protected]> Co-authored-by: svikramjeet <[email protected]> Co-authored-by: Zuber <[email protected]> Co-authored-by: Mohammad Zuber Khan <[email protected]> * Revert "Master -> Dev Merge (#23)" (#24) This reverts commit a875459. * Azure security center enable ddos protection (#27) * Azure security center enable ddos protection remediation script * Removed .DS_Store file * Made changes in README.md * Addressed the review comments * Pre-commit changes added * Made changes in README.md * Added the unit test in tox.ini * fix requirements-dev.txt for tox run Co-authored-by: Mohammad Zuber Khan <[email protected]> Co-authored-by: Shrutika Kulkarni <[email protected]> * Azure Storage default network access deny remediation script (#30) * Azure Storage default network access deny remediation script * .DS_Store file removed * Fixed some logging messages * Added unit test in tox.ini * Changed requirements.txt * Fixed requirements-dev.txt * Pla 19713: remove unused logcall (#31) * remove unused logcall function Co-authored-by: Mohammad Zuber Khan <[email protected]> * Add remediation job to enable ELB access logs (#29) * Add remediation job to enable ELB access logs Co-authored-by: Mohammad Zuber Khan <[email protected]> * PLA-21409: Azure SQL Server Threat Detection enable (#36) * PLA-21822 fixing whitespace (#37) * Pla 21822 fixing requirements whitespace (#38) * PLA-21822 fixing whitespace Co-authored-by: Paul Allen <[email protected]> Co-authored-by: svikramjeet <[email protected]> Co-authored-by: Zuber <[email protected]> Co-authored-by: Mohammad Zuber Khan <[email protected]> Co-authored-by: kshrutik <[email protected]> Co-authored-by: Shrutika Kulkarni <[email protected]>
vikramsinghvirdi
requested review from
regaw-leinad,
nandeshguru and
pauldallen
December 21, 2020 17:52
nandeshguru
approved these changes
Jan 5, 2021
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PLA-17940 updating constraints and fixing s3_enable_access_logging to… (PLA-17940 updating constraints and fixing s3_enable_access_logging to… #3)
PLA-17940 updating constraints and fixing s3_enable_access_logging to not remediate the logging target bucket
PLA-17940 fixing copyright info
Updated readme to have link for reporting issues (Updated readme to have link for reporting issues #4)
Co-authored-by: svikramjeet [email protected]
S3 access logs permissions (S3 access logs permissions #6)
Update minimum permissions for the job
Add more logs when permission is missing
Co-authored-by: Mohammad Zuber Khan [email protected]
Co-authored-by: Mohammad Zuber Khan [email protected]
PLA-16779: Add remediation job for azure security group port 22 (PLA-16779: Add remediation job for azure security group port 22 #10)
PLA-16779: Add remediation job for azure security group port 22
add test dependencies
update README.md
add deployment info to the README
add rule information
Co-authored-by: Mohammad Zuber Khan [email protected]
Add remediation job for closing port 22 for VM (Add remediation job for closing port 22 for VM #11)
PLA-18743: Add remediation job for closing port 22 for VM
add tests for azure_vm_close_port_22 to tox
add deployment details
Co-authored-by: Mohammad Zuber Khan [email protected]
Add remediation jobs for storage and RDP violations (Add remediation jobs for storage and RDP violations #12)
add remediation job for closing RDP access
Add remediation job for remove public access for blob
Add remediation job to allow only https traffic to storage account
update READMEs to fix broken links
Co-authored-by: Mohammad Zuber Khan [email protected]
Co-authored-by: Mohammad Zuber Khan [email protected]
Co-authored-by: Mohammad Zuber Khan [email protected]
Fix ports range for network security groups (Fix ports range for network security groups #19)
handle the case when the security rule port is a range
Add minimum permissions for each remediation jobs
add link to built in roles
Co-authored-by: Mohammad Zuber Khan [email protected]
Co-authored-by: Mohammad Zuber Khan [email protected]
PLA-20459: Add rule information for the remediation job (PLA-20459: Add rule information for the remediation job #22)
update the remediation job payload (Update remediation job payload #21)
Co-authored-by: Mohammad Zuber Khan [email protected]
PLA-20459: Add rule information for the remediation job
add another rule remediated by the job
Co-authored-by: Mohammad Zuber Khan [email protected]
Master -> Dev Merge (Master -> Dev Merge #23)
Release/v1.0.0 (Release/v1.0.0 #7)
PLA-17940 updating constraints and fixing s3_enable_access_logging to… (PLA-17940 updating constraints and fixing s3_enable_access_logging to… #3)
PLA-17940 updating constraints and fixing s3_enable_access_logging to not remediate the logging target bucket
PLA-17940 fixing copyright info
Updated readme to have link for reporting issues (Updated readme to have link for reporting issues #4)
Co-authored-by: svikramjeet [email protected]
S3 access logs permissions (S3 access logs permissions #6)
Update minimum permissions for the job
Add more logs when permission is missing
Co-authored-by: Mohammad Zuber Khan [email protected]
Co-authored-by: Paul Allen [email protected]
Co-authored-by: svikramjeet [email protected]
Co-authored-by: Zuber [email protected]
Co-authored-by: Mohammad Zuber Khan [email protected]
Release/v1.1.0 (Release/v1.1.0 #17)
PLA-17940 updating constraints and fixing s3_enable_access_logging to… (PLA-17940 updating constraints and fixing s3_enable_access_logging to… #3)
PLA-17940 updating constraints and fixing s3_enable_access_logging to not remediate the logging target bucket
PLA-17940 fixing copyright info
Updated readme to have link for reporting issues (Updated readme to have link for reporting issues #4)
Co-authored-by: svikramjeet [email protected]
S3 access logs permissions (S3 access logs permissions #6)
Update minimum permissions for the job
Add more logs when permission is missing
Co-authored-by: Mohammad Zuber Khan [email protected]
Co-authored-by: Mohammad Zuber Khan [email protected]
PLA-16779: Add remediation job for azure security group port 22 (PLA-16779: Add remediation job for azure security group port 22 #10)
PLA-16779: Add remediation job for azure security group port 22
add test dependencies
update README.md
add deployment info to the README
add rule information
Co-authored-by: Mohammad Zuber Khan [email protected]
Add remediation job for closing port 22 for VM (Add remediation job for closing port 22 for VM #11)
PLA-18743: Add remediation job for closing port 22 for VM
add tests for azure_vm_close_port_22 to tox
add deployment details
Co-authored-by: Mohammad Zuber Khan [email protected]
Add remediation jobs for storage and RDP violations (Add remediation jobs for storage and RDP violations #12)
add remediation job for closing RDP access
Add remediation job for remove public access for blob
Add remediation job to allow only https traffic to storage account
update READMEs to fix broken links
Co-authored-by: Mohammad Zuber Khan [email protected]
Co-authored-by: Mohammad Zuber Khan [email protected]
Co-authored-by: Mohammad Zuber Khan [email protected]
Fix ports range for network security groups (Fix ports range for network security groups #19) (Fix ports range for network security groups (#19) #20)
handle the case when the security rule port is a range
Add minimum permissions for each remediation jobs
add link to built in roles
Co-authored-by: Mohammad Zuber Khan [email protected]
Co-authored-by: Zuber [email protected]
Co-authored-by: Mohammad Zuber Khan [email protected]
Co-authored-by: Paul Allen [email protected]
Co-authored-by: svikramjeet [email protected]
Co-authored-by: Zuber [email protected]
Co-authored-by: Mohammad Zuber Khan [email protected]
Co-authored-by: Paul Allen [email protected]
Co-authored-by: svikramjeet [email protected]
Co-authored-by: Zuber [email protected]
Co-authored-by: Mohammad Zuber Khan [email protected]
This reverts commit a875459.
Azure security center enable ddos protection (Azure security center enable ddos protection #27)
Azure security center enable ddos protection remediation script
Removed .DS_Store file
Made changes in README.md
Addressed the review comments
Pre-commit changes added
Made changes in README.md
Added the unit test in tox.ini
fix requirements-dev.txt for tox run
Co-authored-by: Mohammad Zuber Khan [email protected]
Co-authored-by: Shrutika Kulkarni [email protected]
Azure Storage default network access deny remediation script (Azure Storage default network access deny remediation script #30)
Azure Storage default network access deny remediation script
.DS_Store file removed
Fixed some logging messages
Added unit test in tox.ini
Changed requirements.txt
Fixed requirements-dev.txt
Pla 19713: remove unused logcall (Pla 19713: remove unused logcall #31)
remove unused logcall function
Co-authored-by: Mohammad Zuber Khan [email protected]
Add remediation job to enable ELB access logs (Add remediation job to enable ELB access logs #29)
Add remediation job to enable ELB access logs
Co-authored-by: Mohammad Zuber Khan [email protected]
PLA-21409: Azure SQL Server Threat Detection enable (PLA-21409: Azure SQL Server Threat Detection enable #36)
PLA-21822 fixing whitespace (PLA-21822 fixing whitespace #37)
Pla 21822 fixing requirements whitespace (Pla 21822 fixing requirements whitespace #38)
PLA-21822 fixing whitespace
Co-authored-by: Paul Allen [email protected]
Co-authored-by: svikramjeet [email protected]
Co-authored-by: Zuber [email protected]
Co-authored-by: Mohammad Zuber Khan [email protected]
Co-authored-by: kshrutik [email protected]
Co-authored-by: Shrutika Kulkarni [email protected]