[PR #4042] [MERGED] HSB-431 fix: email case sensitive in email provider #4636

New issue

Closed

opened 2026-03-17 02:09:23 +03:00 by kerem · 0 comments

kerem commented

2026-03-17 02:09:23 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/hoppscotch/hoppscotch/pull/4042
Author: @mirarifhasan
Created: 5/7/2024
Status: ✅ Merged
Merged: 5/15/2024
Merged by: @balub

Base: patch ← Head: fix/email-case-sensitive

📝 Commits (3)

6f23667 feat: code level email insensitivity added
480a5f1 test: fix broken test case
5d05518 chore: updated testcase for findUserByEmail in user module

📊 Changes

6 files changed (+34 additions, -20 deletions)

View changed files

📝 packages/hoppscotch-backend/src/admin/admin.service.spec.ts (+5 -1)
📝 packages/hoppscotch-backend/src/admin/admin.service.ts (+9 -3)
📝 packages/hoppscotch-backend/src/shortcode/shortcode.service.ts (+4 -1)
📝 packages/hoppscotch-backend/src/team-invitation/team-invitation.service.ts (+5 -4)
📝 packages/hoppscotch-backend/src/user/user.service.spec.ts (+2 -2)
📝 packages/hoppscotch-backend/src/user/user.service.ts (+9 -9)

📄 Description

Closes HSB-431
Closes #4027

Description

This pull request addresses a problem where user emails are treated as case-sensitive during signup. This means that signing up with user@gmail.com and USER@gmail.com would create two separate user entries in the database.

Solution

This PR implements logic to convert all email addresses to lowercase before saving them in the database. This ensures that regardless of the case used during signup, only one user entry will be created for the same email address.

Checks

My pull request adheres to the code style of this project
My code requires changes to the documentation
I have updated the documentation as required
All the tests have passed

Additional Information

Nil

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/hoppscotch/hoppscotch/pull/4042 **Author:** [@mirarifhasan](https://github.com/mirarifhasan) **Created:** 5/7/2024 **Status:** ✅ Merged **Merged:** 5/15/2024 **Merged by:** [@balub](https://github.com/balub) **Base:** `patch` ← **Head:** `fix/email-case-sensitive` --- ### 📝 Commits (3) - [`6f23667`](https://github.com/hoppscotch/hoppscotch/commit/6f23667c9cecf0896c06963af7706ed38f2eb914) feat: code level email insensitivity added - [`480a5f1`](https://github.com/hoppscotch/hoppscotch/commit/480a5f1f79f997ee9dea3c42feb3c689010e4bf7) test: fix broken test case - [`5d05518`](https://github.com/hoppscotch/hoppscotch/commit/5d05518024009a0a15968b441c8cb839b9b86c1a) chore: updated testcase for findUserByEmail in user module ### 📊 Changes **6 files changed** (+34 additions, -20 deletions) <details> <summary>View changed files</summary> 📝 `packages/hoppscotch-backend/src/admin/admin.service.spec.ts` (+5 -1) 📝 `packages/hoppscotch-backend/src/admin/admin.service.ts` (+9 -3) 📝 `packages/hoppscotch-backend/src/shortcode/shortcode.service.ts` (+4 -1) 📝 `packages/hoppscotch-backend/src/team-invitation/team-invitation.service.ts` (+5 -4) 📝 `packages/hoppscotch-backend/src/user/user.service.spec.ts` (+2 -2) 📝 `packages/hoppscotch-backend/src/user/user.service.ts` (+9 -9) </details> ### 📄 Description    Closes HSB-431 Closes #4027 ### Description  This pull request addresses a problem where user emails are treated as case-sensitive during signup. This means that signing up with `user@gmail.com` and `USER@gmail.com` would create two separate user entries in the database. #### Solution This PR implements logic to convert all email addresses to lowercase before saving them in the database. This ensures that regardless of the case used during signup, only one user entry will be created for the same email address.  ### Checks  - [x] My pull request adheres to the code style of this project - [ ] My code requires changes to the documentation - [ ] I have updated the documentation as required - [x] All the tests have passed ### Additional Information  Nil --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>