dehtml() function prints an AttributeError in Python 3 about instapaper HOT 3 CLOSED

I think it only prints the exception instead of raising it because it still returns the original text. Of course that could be changed if that's bad behavior.

I've pushed a potential fix for this to the "issue-6" branch, try it out let me know.

from instapaper.

Hmm. If I run my example program on Python 3, I get a different error now:

$ python demo.py
Traceback (most recent call last):
  File "/Users/alexwlchan/repos/instapaper/instapaper.py", line 113, in dehtml
    text = str(text, 'UTF-8')
TypeError: decoding str is not supported

I think the correct behaviour here is to pass through text unmodified – i.e., the patch should be

if sys.version_info < (3, 0):
    text = text.decode('utf-8')

In Python 2, text is a bytestring, so we need to decode it to Unicode before passing it up the stack. But Python 3 is dealing with Unicode strings natively, so it’s already been decoded for us.

from instapaper.

I confirm that the fix from @alexwlchan works as expected. Please see PR #8 for an updated version.

from instapaper.