Trim White space, sanitize url

[StrawberryCalpico]

Description

Related PRS (if any):

N/A

Main changes explained:

Add third party library to sanitize and trim urls received by userprofileController

How to test:

1. check into current branch
2. do npm install and ... to run this PR locally
3. use postman to test the api http://localhost:4500/api/userProfile/ {your user id here}
4. check that white space in user lastname and firstname are ignored
5. check that white space in url are also ignored
6. check that empty url name will receive an error

Screenshots or videos of changes:

Before:

Bug.mp4

After:

FixNameWhiteSpace.mp4

FixUrl.mp4

Notes

can get the payload of the http request from chrome developer tools by opening the network tab, send an request, and in the payload section, click view source. Then copy it and paste it into postman body->raw

[palakgosalia]

Hey, I reviewed your PR and also tested it. It works as intended. I tried to add space between the name and URL and after saving and refreshing, it removed all the white spaces. I also tried the same using POST put and get request and was able to see the same results. Great work. Thank you.

601.mp4

[KurtisIvey]

Great job on making the backend a little more robust. It's a welcomed improvement for sure on ensuring proper data validation. I just had some small suggestions as I work down from the userProfileRouter.js

/userProfile

• I wouldn't recommend implementing that .replace method here that removes every bit of whitespace as some users may have two parts to their first name or last name such as "Bobby Joe". This .replace would turn that into BobbyJoe. However, I definitely understand the intention, and would instead recommend using .trim(). This will remove any unnecessary leading or trailing whitespace without removing any intended whitespace.

/userProfile/:userId

• same thing as the suggestion noted above but only applying it to "firstName" and "lastName".
• I think the custom sanitizer is a great use for the personal links, but I would suggest removing the .replace(/\s/g, "") from the "Name" variable and instead opting for a .trim() due to some websites having multiple words in them. Keep the .replace in the link though as that is a fantastic implementation.

Other than that, the linter is lighting up like a christmas tree and customSanitizer is an unused variable that can be removed since it's chained to .body.

Apologies for the background noise, my wife is watching House of villains lol...
https://github.com/OneCommunityGlobal/HGNRest/assets/19482363/b186eba6-7139-46ee-8fee-1fab73f82528