Updates to earlier fixes to address outage logs #3098
Merged
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.
Progress towards #3096
Changes:
serializeUser()
inserver/config/passport.js
async/await
syntax and adds more specified error messaging inserver/controllers/aws.controller.js
andserver/controllers/project.controller/deleteProject.js
server/controllers/project.controller/__test__/deleteProject.test.js
with changes todeleteProject.js
console.error()
in ``server/models/user.js`. This scenario happens when a user who signed up via Google or Github attempts to login through email/password. A better way to handle this can be revisited.authenticateOauth()
function to create route handlers for each login service and adds error handling when no user is present inserver/routes/passport.routes.js
I have verified that this pull request:
npm run lint
)npm run test
)develop
branch.Fixes #123