Skip to content

Stage to Prod#125

Merged
MashB merged 4 commits intomasterfrom
stage
Mar 31, 2025
Merged

Stage to Prod#125
MashB merged 4 commits intomasterfrom
stage

Conversation

@MashB
Copy link
Copy Markdown
Collaborator

@MashB MashB commented Mar 31, 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 31, 2025 05:52
@MashB MashB requested a review from shweta2101 March 31, 2025 05:57
@MashB MashB merged commit c689e4a into master Mar 31, 2025
5 of 7 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.

4 participants