What is your hint or solution suggestion?
The tests validate that you have added the code in your server.js
file for the two routes. They expect the route to be defined using app.route(...).get(...)
syntax where the route
and get
calls are on the same line, such as
app.route(<insert path here >).get(<insert passport.authenticate()
call here>
If you are defining the route directly via get call, it will not work :
app.get(<insert path here >, <insert passport.authenticate()
call here>
Challenge: Implementation of Social Authentication
Link to the challenge:
Learn to code. Build projects. Earn certifications.Since 2015, 40,000 graduates have gotten jobs at tech companies including Google, Apple, Amazon, and Microsoft.
Sky020
September 24, 2020, 4:49pm
2
Thank you, for mentioning this.
I have created an issue for this:
opened 04:44PM - 24 Sep 20 UTC
The tests need to be updated to allow for the two main ways of defining routes:
app.get('/auth/github/callback')
.post(passport.authenticate('local', { failureRedirect: '/' }),...
help wanted
scope: learn
As such, I do not think there is a need to add this to the guide. Hopefully, should be fixed soon.