Fix #99 - Let external user to decide the option handle_failures

[johnklee]

This PR is a candidate solution to solve issue #99 which is about exception caused by python-readability owing to invalid process on IPv6 address. The key idea is to let external user of python-readability to setup option handle_failures='discard' (which currently is invisible to external user) to get around issue #99.

[johnklee]

Hi @buriy
Many thanks for your time and great help in review this PR. I tried to update package readability-lxml

$ pip install --upgrade readability-lxml
Requirement already up-to-date: readability-lxml in ./env/lib/python3.6/site-packages (0.7)

It seems that the pypi has not been updated yet. Can you help to confirm it?
Many thanks for this great repo and your help again.

[buriy]

Yes, I sometimes check the tests and publish it.
But you can also do pip install from github repo.

[johnklee]

Hi @buriy
Sorry to bother you again. I have been checking pypi for a few days. Currently we adopted the temporary solution you suggest to install from github repo locally which is ok for testing. However, our pipeline will do installation each time during deployment by pip install -r requirements.txt. So it will be helpful to have an updated version in pypi to avoid subtle mistakes which may lead to unexpected bug in the pipeline. So is it possible to know when will you have new release in pypi? So we can update the requirements.txt as well to reflect the change?

Many thanks!

[buriy]

Ok, I'll publish it this week, ASAP, and will I'll let you know.

[johnklee]

Hi @buriy
Any update for the next version of readability-lxml (currently, is still 0.7)? Sorry for the bothering and thanks in advance for the help.

[buriy]

Tested and published.

…

On Mon, Apr 29, 2019 at 7:24 AM John ***@***.***> wrote: Hi @buriy <https://github.com/buriy> Any update for the next version of readability-lxml <https://pypi.org/project/readability-lxml/> (currently, is still 0.7)? Sorry for the bothering and thanks in advance for the help. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#115 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAAFFBGL32A62R76FJIEB5TPSY543ANCNFSM4HGYYQ6Q> .

-- Best regards, Yuri V. Baburov, Skype: yuri.baburov