feat(email-verification): send real email + Swagger servers (:3000/api) #149
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ration Made changes on python file
…e/mealplan WIP meal plan api get post delete
Added Swagger UI to project (automatically opens on launch) to assist frontend in using the endpoints. Removed postman files.
…e/api-spec-swagger Updated OpenAPI Spec + Swagger UI
…e/fix-user-prefs Changed API spec for user preferences to be POST instead of PUT
API-Patch notes
Development of logging file
using mocha and chai wrote unit test for login endpoint
Developed Vulnerability tool
…ability Create Vulnerability_V1.0.py
some login api tests complete: - still need to add tests for MFA enabled log ins - need to refactor and use stubs as well, don't want to use real credentials for tests sign up API tests done fixed bug in signupController that allowed duplicate usernames to be created in the DB: - userExists.username doesnt work because getUser returns an array, changed to userExists.length > 0 so it checks if anything is in the array - now wont be able to create users with duplicate usernames, whereas could before added in a deleteUser function for the DB also changed getUser to get entire user row from DB instead of just username, we can probably remove the getUserCredentials function entirely and just use getUser
appointment api tests written also added deleteAppointment function
…e/recipe-image Added recipe image saving and retrieval
contact us tests done recipe tests done but found some bugs that i need to come back to to fix - add recipe isnt adding to the relations table
(API): Test (function) - (Expected Result)
…e/recipe-image bugfix if image isnt included in the payload
…esting-eric Unit testing
32 test cases all passing also fixed a few bugs that I found while writing the tests
…esting integration testing
V1.1 Version with 5 Additional Vulnerability Patterns Identified
… using separated module
added audit-logs for user logins and account creation
…-estimation Recipe Cost Estimation API: Fix some issues (error handling & SwaggerUI) and complete enhancement
…_Cyber_Failed_Login_Email Implemented Failed Login Email Alerts
…ad-protection Implement Role-Based Access Control (RBAC) with JWT Role Injection and Middleware
…_signupfunctionality_integrationteam Signup Functionality (Integration Team)
…th controller update and docs
…pabase admin client and docs
…e routes/verify.js and logs
Contributor
|
Hi, your PR has too many conflicts that must be resolved. Please only focus on the main changes of your features only, don't overwrite other's code. Thank you! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
19 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
(how to test):
) npm ci
2) cp .env.example .env (MAILER_DRIVER=log by default)
3) PORT=3000 npm start
4) POST /api/auth/request-email-verification {"email":"testuser@example.com"}
-> 200 and dev verification link in server logs