[#82] Add email and password pair analyzer #87
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Pull Request Test Coverage Report for Build 170
💛 - Coveralls |
added 2 commits
October 4, 2019 10:38
|
Great work. Will have a closer look tonight! Thank you. |
Implement TwitterAction
Phone number analyzer d-Rickyy-b#90
Database dump analyzer Fixes
Improved the phone number analyzer and added test cases for it.
d-Rickyy-b
requested changes
Oct 5, 2019
| # -*- coding: utf-8 -*- | ||
| from .regexanalyzer import RegexAnalyzer | ||
|
|
||
| _EMAIL_PASSWORD_REGEX = r'[\w\.\+_-]+@[\w\._-]+\.[a-zA-Z]*\:[\w\.\+\!\$\#\^&\*\(\)\{\}\[\]\_\-\@\%\=]+$' |
There was a problem hiding this comment.
Nice regex, but there are some characters which are often used in passwords such as '§´/\?<>;:|",~ and the backtick character. Maybe you could add them to the regex as well? How should we deal with spaces? I would say we ignore spaces for now.
An example of a possible regex: \§\\\/\'\`\´\?\<\>\;\"\:\|\,\~
There was a problem hiding this comment.
I am ignoring the spaces for now and updating the password regex
Add TwitterAction by psidex
Add unit test for SHAHashAnalyzer
I checked this regex against a pastebin with origin and steam keys in it. It matches on Origin keys.
This is necessary for d-Rickyy-b#109
Added Origin key analyzer
Adds easier templating and fixes 'None' values
Implement Battle.Net Key Analyzer
update code
Microsoft License key analyzer.
Microsoft key analyzer tests
Update init.
Add Microsoft key analyzer
Uses websockets to open a connection to Discord's Gateway, then sends the proper identification payload so that further requests are authorized. This code runs only if sending messages fails with code 40001.
That way it only needs to be imported when actually using the bot functionality
I guess this happens for multi threaded programs. No idea how this performs in the long run. Longest test has been 2 minutes and it worked for that time at least.
…cated Add Discord action (webhook or bot token)
Discord action bot
Created basic Epic Key analyzer with unit test fixes d-Rickyy-b#97
Repository owner
deleted a comment
Oct 7, 2019
d-Rickyy-b
approved these changes
Oct 7, 2019
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
My first PR, please let me know if I am missing something.