Deprecated dependencies

[aravindvnair99]

$ npm i -g htmlhint@latest
npm WARN deprecated har-validator@5.1.5: this library is no longer supported
npm WARN deprecated request@2.88.2: request has been deprecated, see https://github.com/request/request/issues/3142

[thedaviddias]

Hey @aravindvnair99, thanks for pointing that out. The repo is going to be updated in the coming days. Hopefully, you will not see these messages in the near future.

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[aravindvnair99]

Hey @aravindvnair99, thanks for pointing that out. The repo is going to be updated in the coming days. Hopefully, you will not see these messages in the near future.

@thedaviddias Any updates?

[aravindvnair99]

As of today, 25th July 2021:

$ npm i -g htmlhint@latest
npm WARN deprecated har-validator@5.1.5: this library is no longer supported
npm WARN deprecated uuid@3.4.0: Please upgrade  to version 7 or higher.  Older versions may use Math.random() in certain circumstances, which is known to be problematic.  See https://v8.dev/blog/math-random for details.
npm WARN deprecated request@2.88.2: request has been deprecated, see https://github.com/request/request/issues/3142

added 75 packages, and audited 76 packages in 11s

5 packages are looking for funding
  run `npm fund` for details

4 moderate severity vulnerabilities

To address all issues (including breaking changes), run:
  npm audit fix --force

Run `npm audit` for details.

Version is 0.15.1

[stuartbale]

request is used in one place - the HtmlHint CLI.

HTMLHint/src/cli/htmlhint.ts

Line 497 in ebaea6c

request.get(url, (error, response, body) => {

If someone can post what the suggested replacement for this is, I can submit a pull request.

Replacing request will resolve all three deprecated warnings.

[stuartbale]

request is used in one place - the HtmlHint CLI.

HTMLHint/src/cli/htmlhint.ts

Line 497 in ebaea6c

request.get(url, (error, response, body) => {

If someone can post what the suggested replacement for this is, I can submit a pull request.

Replacing request will resolve all three deprecated warnings.

Would node-fetch be suitable?

https://www.npmjs.com/package/node-fetch

Seems to be well supported and used in >21k other packages.

[stuartbale]

I notice a pending Pull Request with a fix for this here:
#670

[darcyparker]

with #670 merged, can this be closed?