solution #326
solution #326
Conversation
There was a problem hiding this comment.
Great job on implementing the server functionality! 🎉 Your code meets the task requirements and constraints effectively. The server processes the request URL and query parameters correctly, returning the expected JSON object. While the use of replaceAll('//', '/') might not be necessary and could potentially alter URLs with intentional double slashes, it doesn't critically impact the current functionality. Keep up the good work, and consider revisiting this part to ensure it aligns with your intended use cases. Happy coding! 😊
✨ Please rate this review in the Platform's notification center. I will be grateful for your feedback! ✨
| // Return instance of http.Server class | ||
| return http.createServer((req, res) => { | ||
| const requestURL = new URL( | ||
| req.url.replaceAll('//', '/'), |
There was a problem hiding this comment.
The use of replaceAll('//', '/') may not be necessary or could lead to unexpected results if the URL contains double slashes intentionally. Consider whether this replacement is needed or if it could be handled differently to avoid altering the URL in unintended ways.
No description provided.