extractus / article-extractor

To extract main article from given URL with Node.js
https://extractor-demos.pages.dev/article-extractor
MIT License
1.46k stars 132 forks source link

Add favicon to meta data #350

Closed LarchLiu closed 1 year ago

LarchLiu commented 1 year ago

In some cases, I might want to retrieve the favicon of a website, so I add it.

LarchLiu commented 1 year ago

I checked this issue #120. As you may know, in China, we do not have access to Google. Therefore, I would like to add this feature if it is possible.

ndaidong commented 1 year ago

@LarchLiu thank you, let me see the pr.

ndaidong commented 1 year ago

@LarchLiu I've released v7.2.17 with your pr. Thank you for your contributions.

LarchLiu commented 1 year ago

@ndaidong Thanks a lot. It is very helpful to me.

coveralls commented 5 months ago

Pull Request Test Coverage Report for Build 5418934645

Warning: This coverage report may be inaccurate.

We've detected an issue with your CI configuration that might affect the accuracy of this pull request's coverage report. To ensure accuracy in future PRs, please see these guidelines. A quick fix for this PR: rebase it; your next report should be accurate.


Totals Coverage Status
Change from base Build 5428376447: 0.08%
Covered Lines: 242
Relevant Lines: 243

💛 - Coveralls