Passport: Library still mantained?
I want to use this for my app but i looked into the documentation and is outdated, specifically on the configure middleware part. Is still being mantained? Does it work on Express 4?
DiegoGallegos4
All 12 comments
it works
TrejGun
on 10 Mar 2016
Just because it works, doesn't mean its still being maintained. Windows 95 still "works".
SCKelemen
on 11 Mar 2016
What basis do you make that assumption? Have you tried it? Does it work? Do you want commits for the sake of commits?
Sent from my iPhone
On Mar 11, 2016, at 2:26 PM, Samuel Kelemen [email protected] wrote:
Doesn't look like it
—
Reply to this email directly or view it on GitHub.
jaredhanson
on 11 Mar 2016
Jared, you are my spirit animal.
tbjers
on 12 Mar 2016
Mmh. 146 open issues and no commit in a year or so. It's a valid question I'd say.
joerx
on 15 Mar 2016
How many of the issues are people asking for support, rather than reporting an actual issue with the library? Open source provides you a license for free code, not free support.
jaredhanson
on 15 Mar 2016
The question wasn't about support, it was about the project being maintained.
joerx
on 15 Mar 2016
I stand with joerx.
SCKelemen
on 18 Mar 2016
If you have actionable issues that need fixing in the code base for this project, please report them or, if already reported, comment on an existing issue.
jaredhanson
on 18 Mar 2016
And please read How to Use Open Source and Shut the Fuck Up At the Same Time
jaredhanson
on 18 Mar 2016
@jaredhanson I see the question may have been misunderstood. The question was due because on reading the documentation I was using express 4 and some of the docs where from Express 3. Thanks for the clarification. How can we help you with updating the docs?
DiegoGallegos4
on 26 Mar 2016
@DiegoGallegos4 I didn't misunderstand the question. If you find that the documentation is out of date, the title for an issue should not be "Library still mantained?" Something like, "Documentation refers to out-of-date Express 3.x" or something would be more descriptive and helpful.
The docs are mostly here: https://github.com/passport/www.passportjs.org
Feel free to submit updates.
jaredhanson
on 28 Mar 2016
Related issues
ksmithut
·
6Comments
angel1st
·
5Comments
xingrz
·
5Comments
angel1st
·
7Comments
scan
·
7Comments
Most helpful comment
What basis do you make that assumption? Have you tried it? Does it work? Do you want commits for the sake of commits?
Sent from my iPhone