Skip to content
This repository has been archived by the owner on Jul 24, 2019. It is now read-only.

Mark deprecated #825

Closed
wbt opened this issue Jul 22, 2019 · 4 comments
Closed

Mark deprecated #825

wbt opened this issue Jul 22, 2019 · 4 comments

Comments

@wbt
Copy link

wbt commented Jul 22, 2019

Maintenance of the underlying phantomjs library was suspended last year, following an announcement the year before. Nowhere in the readme (republished on npm) is this mentioned, and the opposite impression is given by its first full sentences:

What this installer is really doing is just grabbing a particular "blessed" (by this module) version of Phantom. As new versions of Phantom are released and vetted, this module will be updated accordingly.

The PhantomJS link at the start of the readme points to that project's official website, which opens by saying:

Important: PhantomJS development is suspended until further notice (more details).

The last publish of the library, as noted on the NPM page, was 2 years ago. There are vulnerabilities in dependencies and other issues in functionality.

This package should be marked as deprecated in NPM.
Explanatory text should also be added to the readme.

Doing so will help reduce the number of projects adding this as a dependency and encourage projects already using it to shift to something more maintained, like headless Chrome.

This would take relatively little time from @Medium which is clearly still in operation, and could help reduce the flow of incoming Issues and associated attention requirements. @valueof, do you have permissions to do this?

(My ability to investigate further details is blocked by some serious issues with GitHub going on now, so I'm filing this Issue while it still seems possible to do so.)

@valueof
Copy link
Contributor

valueof commented Jul 22, 2019

Thanks for raising this. Let me see if I can deprecate the package.

@wbt
Copy link
Author

wbt commented Jul 23, 2019

Thanks! If possible, please also do the same with bundled-phantomjs-prebuilt and phantomjs-prebuilt2 and phantomjs-prebuilt-macbre which all point to this GitHub repo.

@juliangruber You may wish to do the same with your fork. (GitHub issues cannot be raised on forks.)

@zeevl Same with your fork!

@zeevl
Copy link

zeevl commented Jul 23, 2019

Done. Thanks for the poke!

@valueof
Copy link
Contributor

valueof commented Jul 23, 2019

I've deprecated our NPM package. Unfortunately, we don't have access to the other three packages since they are maintained by the outside developers. I've also updated the README and going to archive this repo shortly.

Again, thank you for raising this, @wbt, I really appreciate it.

@valueof valueof closed this as completed Jul 23, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants