How to Use Passport Strategies & Create New Middleware

[clickingmouse]

https://learn.freecodecamp.org/information-security-and-quality-assurance/advanced-node-and-express/create-new-middleware/

1. code does not work for mongodb package 3.X needs 2.X or will get db.collection is not a function. hence this dependency needs to be more strictly enforced
   error...

2. test no.2 does not pass even though i am pretty certain code redirects to '/'

Challenge Name

https://beta.freecodecamp.org/en/challenges/advanced-node-and-express/create-new-middleware
https://beta.freecodecamp.org/en/challenges/advanced-node-and-express/how-to-use-passport-strategies

Issue Description

Browser Information

• Browser Name, Version:
• Operating System:
• Mobile, Desktop, or Tablet:

Your Code

https://tan-bow.glitch.me/

Screenshot

[QuincyLarson]

@Em-Ant Please take a look at this. What are your thoughts on this? Is this something that we should fix?

[Em-Ant]

@QuincyLarson we have CORS issues (probably for all the challenges) because https;//learn.freecodecamp.com is no included in the list of allowed domains. We should check all the projects, they have different strategies for handling CORS.

[Em-Ant]

Create new middleware works, allowing the new domain

@clickingmouse be aware that the redirection is tested searching for the text 'Home page' so you have to put it somewhere in the '/' page. Probably the test or the description could be improved.

check this: https://glitch.com/edit/#!/oasis-thief

[QuincyLarson]

@Em-Ant Awesome! Thanks! I'm closing this issue.

@clickingmouse Can you think of a way that we could improve the challenge description so that this is less ambiguous? If so, would you be interested in opening a pull request to improve the wording?

[raditotev]

This is still failing if there is no "Home page" content on the index page.

[Em-Ant]

@raditotev it's how the current test implementation works. I didn't fix that, read my comment above. The simplest solution is to specify that "Home page" is required in the description.

[mattszcz]

This is still an issue. Needs to be re-opened.

Currently, the only way to pass this challenge is to search the FCC forum or issue tracker for this hidden solution.

[ghost]

Still broken for me. I just put "Home Page" in the index.pug description. Either add that to the challenge or just fix the glitch/github clone?

[Em-Ant]

@mdwattendorf @mattszcz you are right. The test works checking that 'Home page' is in the pug template. I think we should update the challenge description specifying this requirement.

[BTmathic]

@Em-Ant Wouldn't it be better for the test to just look at the (string) URL and split it at ".com" to check that you get "" as the second element, instead of searching for a random string on the page?

Searching for a string could fail if the page is ever changed in the future (or as it currently exists) and isn't even testing properly if that string is found on other pages, whereas the URL should always contain the route you are on.

[BTmathic]

(I should add, this is my first time looking at the challenges/tests so I could be missing something obvious about why this can't be done, but it just looks like jQuery with the $.get() so I imagine this would work.)

[iasjem]

Still broken for me. I just put "Home Page" in the index.pug description. Either add that to the challenge or just fix the glitch/github clone?

I also experience the same issue for that part of Create New Middleware. This solution worked for me very well though. Code to add for views/pug/index.pug:

meta(name='description', content='A cool thing made with HyperDev. This is also the Home page')

[SkipHendriks]

I thought I'd go ahead and create a pull request for the clone. No clue if that would automatically propagate to the glitch version, but I do sure hope so. I changed the description to meta(name='description', content='Home page') Changing the test to check for an URL might be more future proof, but I highly doubt we will break the test in the future.

[kingdopamine]

Just wasted hours of my life on this challenge due to the omission of very simple information critical to passing this challenge. How can FCC not include something as simple as "change the title from 'Hello' to 'Home page' in the instructions. What a piss take. I'm furious.

[RandellDawson]

@kingdopamine Would you mind creating a new issue for your complaint. The original problem this issue was created for has been resolved, so to get a fresh pair of eyes on your issue, it is best to create a new issue. Thank you.

[djasuncion]

LMAO! This is my second round on the lessons and I still got stuck on this one. Been months since my first go, we still haven't fixed this. 💃

[DanPete]

Just found the solution from here as well. Had to add 'Home page' to index.pug.

It seems that there still might not be enough direction or the glitch url is cloning an older version of the repo.

[badrabbitcuntpuncher]

So I get theses emails in my junk folder all the time, what is freeCodeCamp? And why is all ur email conversations go threw my spam folder?

…

On Tue, Mar 31, 2020, 10:40 PM Deorabbin Jibe Medina Asuncion < ***@***.***> wrote: LMAO! This is my second round on the lessons and I still got stuck on this one. Been months since my first go, we still haven't fixed this. 💃 — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#16967 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AEJQEDCN6R233DTHCE3HDDLRKLHVTANCNFSM4EXRXNCQ> .

[pabloCabrera7]

The problem is still there, i got stuck quite a long time, hopefully I went to see the forum otherwhise coudnt notice that!
Finally resolve passing the title of Home page...