Removes Passport-Slack due to project abandonment with critical issues #102
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
This MR removes the link to Passport-Slack from the strategies page due to its abandonment with critical issues.
The text of the NPM page, and the strategy page on passportjs.org reads "This repository is not getting the love it deserves. If you're interested in taking over maintenance or ownership of this repo and npm, please reach me (Mike Pearson) at [email protected]" I attempted to contact the original author via email and GitHub issues to no success. Their domain no longer appears to be active, and their profile shows only one commit within the last 6 years, subtly changing thie plea in the R'EADME. (That commit was 2018.)
Due to lack of support and common bugs, a fork of this repo was created and added to Passport's strategy list 5 years ago. However, since the abandoned one shows up first in the list, and has more downloads per week (likely still due to its position), this is the one that new users choose, not knowing until significant time is wasted. (I say this, having recently went through this journey.)
In order to prevent the strategies page from directing people to an inactive project rather than an active project, I propose removal of passport-slack.