-
-
Notifications
You must be signed in to change notification settings - Fork 452
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
API Docs missing some required parameters. #1871
Comments
hey, thanks for reporting. we are having difficulties in properly handling the documentation with ReDoc and drf-spectacular (we want to change them) so there might be some missing things in the docs. We didn't get any problem reported before because usually people leverage the supported clients so that's easier for them. If I may ask, which is your use case that does not allow to use the official libraries? Are you using a different language? Getting this kind of feedback from the users is very important to us and could drive the decision of the mantainers and our roadmap. Meanwhile, for your specific case, you can check the params required by the API for a single analysis which are inherited by the multiple analysis endpoint. Then, the 500 status code is never a good thing (it should have been a 400) so we'll investigate it further. |
Thanks for the update! For our use case, I'm integrating the IntelOwl API into a SIEM/SOAR platform which does not support third-party libraries (technically not true, but to use them I'd have to log a support ticket every time I needed to change/update any libraries, which is impractical). Thanks for the link, I did see this one already - it may be better to use this particular endpoint in hindsight, as I can make the requests separately. Hope that helps clarify the situation. |
Related bug #1742 |
closing this. We plan to refactor the docs to be more easily usable and maintainable: see #2043 |
What happened
The API documentation seems to be missing some required parameters for analysing multiple observables. It seems that the API docs have not been updated to show the correct keys required for the JSON payload.
I am unsure if there are other endpoints that have the same issues, as this is the only one I've tried to configure so far.
Note: I am not using
pyintelowl
orgo-intelowl
, this is just standard HTTP requests to the REST API. In my particular use case, I cannot currently make use of these libraries.Environment
What did you expect to happen
API Docs should show the correct requirements for making an API request to the application for multiple observables to be analysed.
How to reproduce your issue
I wanted to create an API request for analyzing an observable. After seeing that
/api/analyze_multiple_observables
was one of the endpoints I attempted to configure the API request as per the documentation.However, the provided documentation does not actually specify what an observable item should look like, nor does it include keys such as the
tlp
,connectors_requested
andanalyzers_requested
(amongst others) that seem to be required in the request (which I think originated from this issue/PR: #1020 (comment))Without including the correct format for the payload, I receive a
500
, because the format was incorrect.Request:
Response:
With the correct payload, I receive a
200
response from the API stating that the analysis is running.Request:
Response:
Error messages and logs
N/A
The text was updated successfully, but these errors were encountered: