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.

Sign up for GitHub

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

Jump to bottom

Database Dump Analyzer #72

Closed
d-Rickyy-b opened this issue Sep 30, 2019 · 3 comments
Closed

Database Dump Analyzer #72

d-Rickyy-b opened this issue Sep 30, 2019 · 3 comments
Labels
Difficulty: Easy This is an easy issue which should not take a lot of time good first issue Good for newcomers hacktoberfest Label for issues suited for the Hacktoberfest event New Analyzer For suggesting a new analyzer

Comments

@d-Rickyy-b
Copy link
Owner

There should be a new analyzer to find pastes which contain a database dump.

To solve this issue there needs to be a new analyzer added in the analyzers directory. The analyzer must follow the example of the other analyzers in this directory. Don't forget to add the analyzer to the analyzer package file.

The most simple example for such an analyzer is the AlwaysTrueAnalyzer which always returns True.

This analyzer should match different kinds of database dumps. There are different formats for database dumps. It's up to you how many of those you want to implement. On haveibeenpwned.com there is a nice explanation on database dumps in pastes which should serve as a source for ideas.

Tests are not necessary but highly appreciated. If there are questions, don't hesitate to contact me.
@d-Rickyy-b d-Rickyy-b added good first issue Good for newcomers New Analyzer For suggesting a new analyzer hacktoberfest Label for issues suited for the Hacktoberfest event Difficulty: Easy This is an easy issue which should not take a lot of time labels Sep 30, 2019
@trixpan
Copy link

trixpan commented Oct 1, 2019

happy to own this

@d-Rickyy-b
Copy link
Owner Author

Hey @trixpan you are of course free to work on this - BUT there have been two PRs already for this issue (#76 and #75). In order to prevent duplicate work I would suggest picking another issue. All issues I marked as hacktoberfest are pretty easy to implement in no longer than ~30 minutes max.

@d-Rickyy-b
Copy link
Owner Author

Merged #76

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Difficulty: Easy This is an easy issue which should not take a lot of time good first issue Good for newcomers hacktoberfest Label for issues suited for the Hacktoberfest event New Analyzer For suggesting a new analyzer
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@trixpan @d-Rickyy-b