[SHARE-912][Feature]Add JSON encrypted field to sourceconfig - Githubissues

CenterForOpenScience / SHARE

SHARE is building a free, open, data set about research and scholarly activities across their life cycle.

http://share-research.readthedocs.io/en/latest/index.html

Apache License 2.0

101 stars 58 forks source link

[SHARE-912][Feature]Add JSON encrypted field to sourceconfig #688

Closed sheriefvt closed 6 years ago

sheriefvt commented 7 years ago

Ticket

https://openscience.atlassian.net/browse/SHARE-912

Problem

Add a encrypted JSON field to the SourceConfig model for storing sensitive information, like API keys. Why: Whenever a harvester is added that requires API keys, we need to edit the deployments to securely distribute those keys. This will allow us to deploy harvesters faster and reduce the amount of time spent coordinating with devops

Solution:

Add TestEncryptedJsonField to util
Add new field json_encrypt of type TestEncryptedJsonField to the sourceconfig model
Add salt value to share settings
Add tests to the new field type
Add Django migrations

laurenbarker commented 7 years ago

@chrisseto, was this merged via another PR?

chrisseto commented 7 years ago

It was not, it somehow didn't get merged for as part of 2.11 so we bumped it to 2.12

laurenbarker commented 6 years ago

@sheriefvt, would you mind resolving the merge conflicts so we can get this merged?

sheriefvt commented 6 years ago

@laurenbarker sorry forgot about this..will do today

coveralls commented 6 years ago

Changes Unknown when pulling bd00cfa26eba619d2cbb904b9f6ce45c558d3f57 on sheriefvt:feature/SHARE-912 into on CenterForOpenScience:develop.