Updated passport handles expected errors as exceptions

[Kuldeep246]

Added error messages in the third argument of the done callback instead of the first argument in Google and Github sign in features

Fixes #issue-number

Changes:

I have verified that this pull request:

• has no linting errors (npm run lint)
• has no test errors (npm run test)
• is from a uniquely-named feature branch and is up to date with the develop branch.
• is descriptively named and links to an issue number, i.e. Fixes #123

[welcome]

🎉 Thanks for opening this pull request! Please check out our contributing guidelines if you haven't already.

[release-com]

Release Environments

This Environment is provided by Release, learn more!
To see the status of the Environment click on Environment Status below.

• p5.js-web-editor
  • app-ted6f3b.release.editor.p5js.org/
  • preview-ted6f3b.release.editor.p5js.org/

🔧Environment Status : https://app.release.com/public/Processing%20Foundation/env-439bd40bdf

[lindapaiste]

This has some formatting errors which can be fixed by running the lint command.

[lindapaiste]

Thanks for your work on this. I'm going to close this PR and go with #3050 instead because I don't like this particular change. IMO we want to keep the name of the service in there.