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

AWS Secret Key Analyzer #130

Closed
wants to merge 8 commits into from
Closed

AWS Secret Key Analyzer #130

wants to merge 8 commits into from

Conversation

jmmille
Copy link
Contributor

@jmmille jmmille commented Oct 8, 2019

disregard this one. Messed up my branches.

jmmille and others added 8 commits October 6, 2019 20:15
Created md5 hash analyzer unit test.
Regex would trigger on longer SHA hashes.  Added word bounderies to hopefully prevent that.
AWS Keys are 40 character alphanumerics.  Every SHA hash will get picked up as an AWS Key, but AWS keys will rarely get picked up as a SHA hash.
@coveralls
Copy link

coveralls commented Oct 8, 2019

Pull Request Test Coverage Report for Build 185

  • 43 of 44 (97.73%) changed or added relevant lines in 3 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.7%) to 67.469%

Changes Missing Coverage Covered Lines Changed/Added Lines %
pastepwn/analyzers/tests/awssecretkeyanalyzer_test.py 36 37 97.3%
Totals Coverage Status
Change from base Build 184: 0.7%
Covered Lines: 1317
Relevant Lines: 1952

💛 - Coveralls

Repository owner deleted a comment Oct 8, 2019
@jmmille jmmille closed this Oct 8, 2019
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.

None yet

3 participants