-
Notifications
You must be signed in to change notification settings - Fork 577
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 returns 200 for non existing endpoints #3005
Labels
Comments
Working on this issue |
4 tasks
berggren
added a commit
that referenced
this issue
May 7, 2024
* added route error handler * Updating invalid route method with comment, docstring * Fixed invalid api routes to require login * implemented invalid route check and unit test * provided unit test for invalid resources check * Final commit Co-authored-by: Devin Delaney <[email protected]> Co-authored-by: Chynna Syas <[email protected]> Co-authored-by: Dustin Phan <[email protected]> * Update timesketch/app.py Co-authored-by: Janosch <[email protected]> * Update timesketch/app.py Co-authored-by: Janosch <[email protected]> * Update timesketch/app.py Co-authored-by: Janosch <[email protected]> * Update timesketch/app.py Co-authored-by: Janosch <[email protected]> * Update timesketch/app.py Co-authored-by: Janosch <[email protected]> * Update timesketch/api/v1/resources_test.py Co-authored-by: Janosch <[email protected]> * Update app.py for linter compatibility * Update resources_test.py for linter compatibility --------- Co-authored-by: Janosch <[email protected]> Co-authored-by: Johan Berggren <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
With the default set-up the Timesketch API returns an empty page with status code 200 for API endpoints that do not exist.
Reproduce:
Expected behaviour:
The text was updated successfully, but these errors were encountered: