Skip to content

user profile endpoint issue with + character#123

Merged
MashB merged 2 commits intodevfrom
feature-1504
Mar 24, 2025
Merged

user profile endpoint issue with + character#123
MashB merged 2 commits intodevfrom
feature-1504

Conversation

@MashB
Copy link
Copy Markdown
Collaborator

@MashB MashB commented Mar 24, 2025

Bug Fix

DevBoard Task

https://dev.azure.com/TDEI-UW/TDEI/_workitems/edit/1789

Issue Summary

  • User name having with '+' sign, when requests for user profile details , backend service gives 404 user not found.
  • Nodejs considered '+' sign as special character and replaces with space, which is the reason system gives 404.

Fix Implemented

  • Express request.query transforms '+' sign to space, so to avoiding the transform , we used raw query to extract the user_name and decodeURL
  • Fix can take encoded or decoded value and process both of them properly.
  • Ex . username=alex+dev@test.com or alex%2Bdev%40test.com
  • Added extra check for early exit of function if no auth token.

Impacted Areas for Testing

  • Get user profile

@MashB MashB requested a review from a team as a code owner March 24, 2025 07:33
@MashB MashB merged commit 5e6f898 into dev Mar 24, 2025
3 of 5 checks passed
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.

3 participants