Skip to content

Conversation

@Day-fit
Copy link
Owner

@Day-fit Day-fit commented Sep 16, 2025

This pull request introduces a significant refactor to the user identification system, transitioning from using usernames and numeric IDs to universally unique identifiers (UUIDs) throughout the authentication and user management services. The changes impact entity models, repositories, authentication flows, and event handling, ensuring greater consistency and scalability in user identification.

Core changes to user identification:

  • The EncryptifyUser entity now uses a UUID as its primary key instead of a Long, and all related repository interfaces and service methods have been updated to use UUID for user identification. [1] [2]
  • Authentication and principal classes (JwtAuthenticationProvider, UserDetailsImpl, UserPrincipal) have been updated to use UUID for user IDs instead of usernames or numeric IDs, including token subject extraction and principal name representation. [1] [2] [3] [4]

Service and controller updates:

  • User cache and service methods now operate on UUID instead of usernames, including account deletion, fetching user info, and checking if a user is banned. Controllers have been updated accordingly to use the new API. [1] [2] [3] [4] [5]
  • The login and registration flows now return and use UUIDs for user identification in JWTs and cookies, and provide a method for retrieving usernames by UUID. [1] [2] [3] [4]

Event and communication changes:

  • Events and communication services now pass and handle UUIDs for user identification, including the UserReadyForSetupEvent and email verification processes. [1] [2] [3] [4]

Build and configuration updates:

  • The Maven version for encryptify-auth was updated from 0.0.1-SNAPSHOT to 0.0.1.
  • The .idea/compiler.xml was updated to include the encryptify-stats module and to rename an annotation profile. [1] [2]

These changes collectively improve the robustness and future-proofing of user identification across the authentication system.

Day-fit and others added 4 commits September 16, 2025 20:49
…cation; update method signatures and repository queries accordingly.
…eamDto with consolidated FileSystemStreamRequestDto hierarchy
…ove user data structure handling, added "stats" microservice to the CI / CD workflow
@Day-fit Day-fit requested a review from Copilot September 23, 2025 18:37
@Day-fit Day-fit self-assigned this Sep 23, 2025
@Day-fit Day-fit added enhancement New feature or request CD pipeline Changes apply to CD pipeline CI Pipeline Changes apply to CI pipeline labels Sep 23, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This pull request introduces a statistics module with activity tracking and implements a comprehensive user identification system transition from usernames/numeric IDs to UUIDs. Additionally, it adds dark mode support to the frontend and updates chart versions to stable releases.

  • UUID-based user identification system throughout authentication and core services
  • New statistics service with activity tracking via RabbitMQ streams
  • Dark mode implementation in the frontend with user settings management

Reviewed Changes

Copilot reviewed 103 out of 105 changed files in this pull request and generated 7 comments.

Show a summary per file
File Description
helm/Encryptify-Stats/* Helm chart configuration for new statistics service
encryptify-stats/* Complete statistics service implementation with activity tracking
encryptify-core/* Updated to use UUID user identification and activity stream integration
encryptify-auth/* Migrated from username-based to UUID-based authentication system
encryptify-frontend/* Added dark mode support and enhanced user settings management
helm/*/Chart.yaml Updated chart versions from SNAPSHOT to stable releases
Files not reviewed (2)
  • .idea/compiler.xml: Language not supported
  • .idea/runConfigurations/Run_Application.xml: Language not supported

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@Day-fit Day-fit marked this pull request as ready for review September 23, 2025 19:22
@Day-fit Day-fit merged commit 1dc91ad into main Sep 23, 2025
9 checks passed
@Day-fit Day-fit deleted the Statistics-Implementation branch September 23, 2025 19:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CD pipeline Changes apply to CD pipeline CI Pipeline Changes apply to CI pipeline enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants