Feature/2 feature apikeys - #9
Merged
Merged
Conversation
Closed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This pull request introduces a comprehensive refactor and extension of the API key management system, along with improvements to CI workflows and testing infrastructure. The main changes are the extraction of API key logic into a dedicated service and repository, the addition of unit tests for both API key and profile services, and updates to the database schema and CI configuration.
API Key Management Refactor and Enhancements:
ApiKeysServiceandApiKeyRepositoryto encapsulate all API key operations, including creation, validation, revocation, and regeneration, moving this logic out of theProfileServicefor better separation of concerns. TheApiKeyViewtype and database schema were extended to supportexpiresAtandrevokedAtfields. [1] [2] [3] [4] [5]ProfileService, which now focuses solely on user profile and password management. [1] [2]Testing Improvements:
ApiKeysServiceandProfileServiceusing Vitest, ensuring robust coverage for API key and profile operations. [1] [2]CI and Build System Updates:
developtomain. [1] [2]vitestand related coverage tools topackage.jsonscripts and dev dependencies for running and watching tests. [1] [2] [3]These changes improve maintainability, testability, and scalability of the authentication module, especially around API key lifecycle management.