Skip to content
This repository was archived by the owner on Sep 17, 2021. It is now read-only.

Conversation

@scriptsrc
Copy link
Contributor

@scriptsrc scriptsrc commented Mar 28, 2017

TODO BEFORE RELEASE:

v0.9.0 (2017-04-13)

Important Notes:

  • SECURITY_MONKEY_SETTINGS is no longer a required environment variable.
    • If supplied, security_monkey will respect the variable. Otherwise it will default to env-config/config.py
  • manage.py has been moved inside the package and a monkey alias has been setup.
    • Where you might once call python manage.py <arguments> you will now call monkey <arguments>
  • Documentation has been converted from RST to Markdown.
    • I will no longer be using readthedocs or RST.
    • Quickstart guide has been largely re-written.
    • Quickstart now instructs you to create and use a virtualenv (and how to get supervisor to work with it)
  • This release contains GCP Watcher Support.
  • Additional Permissions Required:
    • ec2:DescribeVpnGateways

Contributors:

scriptsrc and others added 30 commits January 19, 2017 20:32
…stringequalsignorecase

Updating ARN.py to look for StringEqualsIgnoreCase in policy condition blocks
* Make JIRA transitions configurable
* Support JIRA proxy settings
* Support JIRA ticket assignee
Type: generic

Why is this change necessary?
Pytest patching has a known issue where patches are ignored when a class
has already been loaded by any other test running in the same process.
This caused random failures when tests are added that import classes
patched by other tests that used to run correctly.

This change addresses the need by:
Splitting the tests out into directories based on the level in the
application of the components being tested and running each directory
seperatly.

Potential Side Effects:
No known side effects
Type: generic-bugfix

Why is this change necessary?
The Netflix merge introduced a two defects:
1) Did not insure condition_accounts was created before used
2) Moved the arn check inside a condition block

This change addresses the need by:
Move condition_accounts and arn check to cover all paths

Potential Side Effects:
No known side effects
Add to second set of fields in search component
Add ability to press enter to search in search bar component
* Update dev_setup_osx.rst to get it up-to-date

* Update dev_setup_osx doc with Install Python step

Add step before Virtualenv to fix ‘sudo pip install virtualenvwrapper --ignore-installed six’ hack

* Remove ‘--ignore-installed six’ hack

Oops

* Update dev_setup_osx doc with Upgrade Pip step

Remove Install Pip step
…s_references 📦

Remove broken packages link
Remove 'sudo' from 'brew install python'
…entation

Minor reformatting/style changes to Docker docs
Organize tests into directories
Type: generic-bugfix

Why is this change necessary?
We were getting intermittent integrity errors and timeouts when
deleting accounts with many items and issues. This appears to be
caused by SQL Alchemy's method of handling cascading deletes,
which is inefficient and does not appear to handle transactional
locks well, allowing for race conditions.

This change addresses the need by:
Deleting accounts and related records with a raw sql query
Type: generic-bugfix

Why is this change necessary?
Some kms keys are partially visible but throw exceptions when viewing
details. The current code does not handle these cases so we get false
access errors and do not see the keys.

This change addresses the need by:
Handles the access errors and shows partial results.

Potential Side Effects:
No known side effects
Why is this change necessary?
Netflix's direction for unit tests is to use a temporary DB,
but we have been using a DB mock class

This change addresses the need by:
Removing db_mock.py and dependencies and refactoring watcher test setup
into new, base watcher test class.

Potential Side Effects:
None
Type: feature

Why is this change necessary?
As the number of accounts being watched increases, it becomes harder to
manage different environments and which accounts they should be
watching. This service enables an API and command-line tool (through
manage.py) that allows a bulk list of accounts to be enabled or disabled
in one command.
@coveralls
Copy link

Coverage Status

Coverage increased (+10.1%) to 61.502% when pulling 450e698 on develop into 870e18b on master.

Mike Grima and others added 2 commits March 29, 2017 14:09
@coveralls
Copy link

Coverage Status

Coverage increased (+10.1%) to 61.507% when pulling 9a6c849 on develop into 870e18b on master.

* Change nginx conf location

* Change nginx conf location

* Change nginx conf location

* Cleaner sed if no SSL

* Add SESSION_COOKIE_SECURE env

* Create default admin user

* Refactor and clean docker and docker-compose files

* Wait the db

* Update gitignore

* Update docker docs

* Restore SSL in nginx config
@coveralls
Copy link

Coverage Status

Coverage increased (+10.1%) to 61.507% when pulling ecd0926 on develop into 870e18b on master.

@coveralls
Copy link

Coverage Status

Coverage increased (+10.1%) to 61.507% when pulling f94fd12 on develop into 870e18b on master.

* Updating install documentation to simplify.

* Hyperlink test.

* Another test

* Link test

* Move to MD?

* Moving more things to Markdown

* Moving everything to markdown

* Updating docker docs

* Adding IAM GCP instructions.

* Adding RDS Postgres docs

* Adding RDS Postgres docs1

* Adding proper apt-get for installing local postgres.

* Bumping FlaskSecurityFork. Making SECURITY_MONKEY_SETTINGS optional, doc updates

* cli docs and typos

* Fixing config path in sample supervisor config

* doc updates

* Adding thigns back into userguide

* Moving travis to config.py

* Creating log folder

* Travis log permissions

* Updating travis DB name

* Disabling CSRF on travis for our tests

* Fixing travis sed

* Moving readme to markdown. various syntax fixes in docs

* Updating readme.

* Adding table to readme

* Putting waffle/gitter on same line

* Reorganizing README

* Adding link to cloudaux in readme
@coveralls
Copy link

Coverage Status

Coverage increased (+10.09%) to 61.461% when pulling 605b6f0 on develop into 870e18b on master.

* Quickstart updates

Removing lots of `sudo`.
Ran through on a new GCP instance and made some changes for clarity.

* quickstart tweaks

* Updating supervisor virtualenv path

* quickstart updates
@coveralls
Copy link

Coverage Status

Coverage increased (+10.09%) to 61.461% when pulling 9cb374c on develop into 870e18b on master.

@coveralls
Copy link

Coverage Status

Coverage increased (+10.1%) to 61.5% when pulling f96ba77 on develop into 870e18b on master.

* Creating monkey entrypoint to replace python manage.py

* Fixing MD headers

* Adding README links to custom watchers and auditors.

* Updating travis to use monkey commands

* Removing manage.py imports
@coveralls
Copy link

Coverage Status

Coverage increased (+9.02%) to 60.392% when pulling 4a0850f on develop into 870e18b on master.

mcpeak and others added 2 commits April 13, 2017 13:16
* Adding an option to preserve group write permission for log files

This commit adds a new log rotate handler that will allow write
permission for members of the same group.  This is useful in
deployment scenarios where other processes manage logs created
by Security Monkey.  Users can enable this feature by uncommenting
the new handler in the config.

* Fixing bug in group write access preserving rotate handler

This commit fixes a bug in the rotating file handler that
preserves group write permission.

* fixing comma

* Update AUTHORS
*  Added doc on update steps

* Updating the filename and a few other minor things

* Updating title of update document.
@coveralls
Copy link

Coverage Status

Coverage increased (+9.0%) to 60.375% when pulling a335af8 on develop into 870e18b on master.

@coveralls
Copy link

Coverage Status

Coverage increased (+9.0%) to 60.375% when pulling a335af8 on develop into 870e18b on master.

* Adding v0.9.0 changelog

* Bumping version from 0.8.0 to 0.9.0
@scriptsrc scriptsrc changed the title [WIP] Preparation for Release v0.9.0 Release v0.9.0 Apr 14, 2017
@coveralls
Copy link

Coverage Status

Coverage increased (+9.0%) to 60.375% when pulling 3ddf79b on develop into 870e18b on master.

@scriptsrc scriptsrc changed the title Release v0.9.0 Release v0.9.0 "13 Reasons" Apr 14, 2017
@scriptsrc scriptsrc merged commit 38b160e into master Apr 14, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.