Skip to content
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

Modified the Login and Signup functionality to remove case-sensitivity #2121

Closed
wants to merge 1 commit into from

Conversation

mThanuj
Copy link

@mThanuj mThanuj commented Dec 9, 2024

Overview:

This pull request introduces several updates across multiple files, including improvements to the authentication logic, storage handling, and user creation process. Additionally, it updates the package dependencies.

Key Changes:

  • Authentication Service (apps/server/src/auth/auth.service.ts):

    • Fixed an issue in the authentication flow where the identifier is now consistently converted to lowercase before querying the user.
  • Storage Service (apps/server/src/storage/storage.service.ts):

    • Improved the object upload process to the storage service by ensuring correct handling of buffer size.
    • Fixed an issue where the buffer size was not being passed correctly to the storage client, preventing potential upload errors.
  • User Service (apps/server/src/user/user.service.ts):

    • Enhanced the create method to ensure email addresses are always stored in lowercase, preventing case-sensitive mismatches during authentication or other operations.
  • Package Updates (package.json):

    • Updated the Prisma client dependencies to newer versions for improved performance and bug fixes.
    • Added the necessary dependencies for better integration with the current codebase.

Below video shows the website after the changes

1740.mp4

@mThanuj mThanuj closed this by deleting the head repository Dec 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant