Added API: Users and Status/Health Endpoints #117
Conversation
mtthidoteu
commented
Jul 11, 2025
mtthidoteu
commented
- Introduced API key model and related functions for creating, authenticating, and managing API keys.
- Enhanced the settings page to allow users to create, delete, and toggle API keys.
- Updated app to include API for Managing Users, as well as a generic status/health endpoint
- Updated README to document the new health check feature.
- Introduced API key model and related functions for creating, authenticating, and managing API keys. - Updated FastAPI application to include a health check endpoint. - Enhanced the settings page to allow users to create, delete, and toggle API keys. - Updated README to document the new health check feature.
|
I am the developer of Wizarr and having a User endpoint would be great, as I would love to integrate it ! |
|
Looks okay on a first glance, thanks! Will do a more thorough review when I have the time in a few days. Have been considering to support json on basically all endpoints to allow for easier scriptability and compatibility with other apps. Thanks for starting with that. |
There was a problem hiding this comment.
Thanks for the PR. It's a nice addition. But yes, there's a lot of AI slop that is not required. A large part of the PR can be deleted.
I adjusted the API key authentication (api.py, authentication.py), so that it also automatically generates the docs for the api key. Please merge in the changes from the api-authentication-pr117 branch before fixing the other things.
After the points I mentioned, type errors, and formatting is fixed, it should be almost ready to merge.
- Renamed ApiKey model to APIKey for consistency. - Updated references to ApiKey in authentication and settings modules. - Removed deprecated health check test script. - Simplified API routes and removed unused endpoints. - Enhanced API key management in settings with improved UI. - Added comprehensive API documentation for key management and usage. - Removed generate_openapi.py script as it is no longer needed. - Created a new Alembic migration for API key table.
