Make storage pluggable - Githubissues

Netflix-Skunkworks / aardvark

Aardvark is a multi-account AWS IAM Access Advisor API

Apache License 2.0

471 stars 77 forks source link

Make storage pluggable #115

Closed patricksanders closed 4 years ago

patricksanders commented 4 years ago

This PR refactors the codebase to provide a clean interface to storage plugins to allow for more configurable backends, e.g. adding support for storing data in DynamoDB.

This is the first in a series of PRs moving toward a v1.0.0 release. These will be breaking changes.

[x] Format codebase using Black to ensure consistency going forward
[x] Refactor storage logic to decouple from Flask
[x] Change how configuration is done
[x] Provide migration path for new configuration - Deferred to #117

Dreadwall commented 4 years ago

See mixing of type hints and doc strings, might want to decide and commit to using or not using those throughout