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

[Bug] Hashdd not working #931

Closed
hefesita opened this issue Jan 28, 2021 · 3 comments · Fixed by #932
Closed

[Bug] Hashdd not working #931

hefesita opened this issue Jan 28, 2021 · 3 comments · Fixed by #932
Assignees

Comments

@hefesita
Copy link

Describe the bug
Hashdd is not working, either the endpoint is not found or the method is not allowed.

To Reproduce
I have configured Hashdd as a Cortex Analyzer and when I try to use it I have an error:
404 Client Error: NOT FOUND for url: https://api.hashdd.com/detail

I modify the analyzer to point to the endpoint in their API docs (https://www.hashdd.com/#api):
https://api.hashdd.com/v1/detail/hash I still get an error 405 Client Error: METHOD NOT ALLOWED.
Looking at the code, the method used is POST and I think it should be GET.

Work environment

  • Server OS: CentOS 8
  • Cortex version: 3
  • Cortex Analyzer/Responder name: Hashdd_Detail_1_0
  • Cortex Analyzer/Responder version: Version: 1.0
@dadokkio dadokkio self-assigned this Jan 28, 2021
@dadokkio
Copy link
Contributor

I was able to fix login and query..but the return details are now a list instead of a simple json.. tomorrow morning I'll find a good way to update templates and push fix

@dadokkio dadokkio linked a pull request Jan 29, 2021 that will close this issue
@dadokkio
Copy link
Contributor

dadokkio commented Jan 29, 2021

I made a pull in develop if you want to test.
In the template there is now a list of results so I tried to put just common info on top.
image

Trying to put all details was a little unusable:
image

If you have any suggestion let me know.

@hefesita
Copy link
Author

hefesita commented Jan 29, 2021

Thanks! It works!

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 a pull request may close this issue.

2 participants