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

Analyzer - Haveibeenpwned.com Lookups #191

Closed
wants to merge 12 commits into from
Closed

Analyzer - Haveibeenpwned.com Lookups #191

wants to merge 12 commits into from

Conversation

znb
Copy link

@znb znb commented Feb 18, 2018

Hello

This is mostly working.

It's in reference to issue #190

Hopefully someone will find it useful

@znb znb changed the base branch from master to develop February 18, 2018 00:39
@3c7 3c7 added the scope:analyzer Issue is analyzer related label Feb 27, 2018
@3c7
Copy link
Contributor

3c7 commented Apr 6, 2018

Hey @znb how's the progress on the reports? Need help?

@crackytsi
Copy link
Contributor

@3c7 @znb I had to do some modifications to get it working, so I created #367
(TBH: I had no clue how I could suggest modifications to this PR)

HaveIbeenpwned seems to have changed some parts on the API.
Additionally I added a template that I use.

@nadouani nadouani added status:duplicate and removed scope:analyzer Issue is analyzer related status:needs-template Analyzer still needs a template for TheHive labels Nov 29, 2018
@nadouani
Copy link
Contributor

This has been improved with #367 and will be merged into 1.15.0 release

@nadouani nadouani closed this Nov 29, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants