Sweep: add forget password functionality to controllers/authController.js and routes/auth.js

Here's the PR! https://github.com/YaraElmalah/voyageit-node-mvc/pull/6.

⚡ Sweep Free Trial: I used GPT-4 to create this ticket. You have 5 GPT-4 tickets left for the month and 0 for the day. For more GPT-4 tickets, visit our payment portal.To get Sweep to recreate this ticket, leave a comment prefixed with "sweep:" or edit the issue.

Install Sweep Configs: Pull Request

Step 1: 🔍 Code Search

I found the following snippets in your repository. I will now analyze these snippets and come up with a plan.

Some code snippets I looked at (click to expand). If some file is missing from here, you can mention the path in the ticket description.

https://github.com/YaraElmalah/voyageit-node-mvc/blob/08eea03699461eef65ae264e9d3354ff74a078f7/routes/auth.js#L1-L23 https://github.com/YaraElmalah/voyageit-node-mvc/blob/08eea03699461eef65ae264e9d3354ff74a078f7/controllers/auth.js#L1-L69 https://github.com/YaraElmalah/voyageit-node-mvc/blob/08eea03699461eef65ae264e9d3354ff74a078f7/README.md#L1-L79 https://github.com/YaraElmalah/voyageit-node-mvc/blob/08eea03699461eef65ae264e9d3354ff74a078f7/app.js#L1-L64 https://github.com/YaraElmalah/voyageit-node-mvc/blob/08eea03699461eef65ae264e9d3354ff74a078f7/package.json#L1-L40

Step 2: 🧐 Snippet Analysis

From looking through the relevant snippets, I decided to make the following modifications:

File Path	Proposed Changes
`controllers/auth.js`	Add two new functions: getForgotPassword and postForgotPassword. The getForgotPassword function should render the 'forgot password' page. The postForgotPassword function should validate the form submission, generate a unique token, associate it with the user's account, and send an email to the user with a link to reset their password.
`routes/auth.js`	Add two new routes for the 'forgot password' page. The GET route should call the getForgotPassword function, and the POST route should call the postForgotPassword function.
`views/auth/forgotPassword.pug`	Create a new view for the 'forgot password' page. This view should contain a form for the user to enter their email address.

Step 3: 📝 Planning

I have created a plan for writing the pull request. I am now working my plan and coding the required changes to address this issue. Here is the planned pull request:

Add forgot password functionality to controllers/authController.js and routes/auth.js sweep/add-forgot-password-functionality

Description

This PR adds the forgot password functionality to the controllers/authController.js and routes/auth.js files. It introduces new routes and controller functions to handle the 'forgot password' page. The form submission will generate a unique token, associate it with the user's account, and send an email to the user with a link to reset their password.

Summary of Changes

Added new routes for the 'forgot password' page in routes/auth.js

Added new controller functions in controllers/authController.js to handle the 'forgot password' page

Created a new view views/auth/forgotPassword.pug for the 'forgot password' page

Implemented form submission processing in the postForgotPassword function, including generating a unique token, associating it with the user's account, and sending an email to the user with a reset password link

Step 4: ⌨️ Coding

File	Instructions	Progress
`controllers/auth.js`	Add two new functions: getForgotPassword and postForgotPassword. The getForgotPassword function should render the 'forgot password' page. The postForgotPassword function should validate the form submission, generate a unique token, associate it with the user's account, and send an email to the user with a link to reset their password.	✅ Commit `08eea03`
`routes/auth.js`	Add two new routes for the 'forgot password' page. The GET route should call the getForgotPassword function, and the POST route should call the postForgotPassword function.	✅ Commit `d12d3ca`
`views/auth/forgotPassword.pug`	Create a new view for the 'forgot password' page. This view should contain a form for the user to enter their email address.	✅ Commit `0c3c568`	I have finished coding the issue. I am now reviewing it for completeness.

Step 5: 🔁 Code Review

Here are the my self-reviews of my changes at sweep/add-forgot-password-functionality.

Here is the 1st review

The changes you've made are a good start, but there are a few things that need to be addressed:

In controllers/auth.js:

On line 78, the validationResult function is used but it's not imported in this file. You need to add const { validationResult } = require('express-validator'); at the top of the file.

On line 83, the crypto module is used but it's not imported in this file. You need to add const crypto = require('crypto'); at the top of the file.

On line 86, the User model is used but it's not imported in this file. You need to add const User = require('../models/user'); at the top of the file.

On line 97, the smtpTransport object is used but it's not defined or imported in this file. You need to define it or import it from another file.

Please make these changes and push them to the branch. After that, we can proceed with the review.

I finished incorporating these changes.

To recreate the pull request, leave a comment prefixed with "sweep:" or edit the issue. ^{Join Our Discord}

YaraElmalah / voyageit-node-mvc