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

Sql database #528

Closed
wants to merge 3 commits into from
Closed

Sql database #528

wants to merge 3 commits into from

Conversation

katehausladen
Copy link
Collaborator

Whenever you guys get to this--

As discussed in this issue, I added axios post requests to addEvidence.js to help us more effectively use privacy pioneer with our crawler. When sql_db = false (sql_db = false by default), then the only difference to privacy pioneer is that now axios is in its package.json; all other code added won't do anything.

Thanks so much!

As stated in our issue 59 (privacy-tech-lab/gpc-web-crawler#59), it'd be great to be able to use an sql db to store data for our crawling purposes. I implemented one to try it out. The rest api to use it is in our repo. More details in our issue 59.
To prevent this db from interfering with Privacy Pioneer's functionality, I added a toggle to turn post requests to the db on/off. By default, sql_db = false, meaning axios post request will not be sent.
@SebastianZimmeck
Copy link
Member

Great work, @katehausladen!

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.

2 participants