Remove extra DB dependency using a DB abstraction layer

WithSecureOpenSource / mittn

Mittn: Security test tool runner for test automation in CI

Apache License 2.0

195 stars 35 forks source link

Remove extra DB dependency using a DB abstraction layer #11

Closed anttivs closed 9 years ago

anttivs commented 10 years ago

Currently, the tool requires both sqlite and postgres to be installed (or at least their respective Python libraries). The code will be refactored to use a DB abstraction layer that should enable use of any major DB, and cut away a lot of code.

anttivs commented 10 years ago

Probably SQLAlchemy Core. Let's see.

anttivs commented 9 years ago

With moving to SQLAlchemy, this will cause changes to database schema (mainly data types that SQLAchemy chooses to use). If you have existing false positive databases, you'll need to migrate.

anttivs commented 9 years ago

Ok, here's the plan:

I will fix & do other changes scheduled for this quarterly release (except the database abstraction), and then tag a "release". Anyone still depending on current database schema can continue using that (clone and checkout that tag).
I will release the new database stuff into master. All further fixes and development will be on master and not backported to the abovementioned release.
If I have time, I'll whip up a database migration tool.

anttivs commented 9 years ago

Landed on the master. This breaks the database schema compatibility; use tag "v0.1" to check out the latest that used the old schema.