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

Syncing my Fork #2

Merged
merged 1,530 commits into from
Dec 16, 2019
Merged

Syncing my Fork #2

merged 1,530 commits into from
Dec 16, 2019

Conversation

shivank1234
Copy link
Owner

Feature / Fixes

Changes/Fixes the following feature

  • Have you added an explanation of what your changes do and why you'd like to include them?

  • Have you successfully tested your changes locally?

  • Example screenshot/log transcript of the feature working

Changes

ShayNehmad and others added 30 commits October 13, 2019 12:34
…etwork-error

463/hotfix/exception on aws network error
# Conflicts:
#	monkey/common/cloud/aws_instance.py
Monkeyzoo update and config_schema improvements
… deployment_scripts_python3

# Conflicts:
#	deployment_scripts/deploy_linux.sh
#	monkey/monkey_island/linux/run.sh
danielguardicore and others added 28 commits November 28, 2019 12:41
# Conflicts:
#	monkey/monkey_island/cc/services/attack/attack_config.py
Updated the refresh intervals to lower server load.
# Conflicts:
#	monkey/monkey_island/cc/ui/src/components/pages/ZeroTrustReportPage.js
@shivank1234 shivank1234 merged commit 70e4d18 into shivank1234:develop Dec 16, 2019
shivank1234 pushed a commit that referenced this pull request Dec 16, 2019
The rules are now set at this.rules array.
While some of them have two conditions, i.e. Rule #2 shoud check if Conclusive is 0 and Inconclusive > 0, all rules has its own function (formula), which returns true or false.

Eventually, I could shorten variable naming, for example, d_['Conclusive'] to something more prompt, but keeping this helps understand formulas even without referencing to upper comments.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants