Skip to content
This repository was archived by the owner on Sep 27, 2021. It is now read-only.

Add textOnly paramater to get_links_from_email#12

Open
koshie wants to merge 1 commit into
lovelysystems:masterfrom
koshie:GET_TEXT_ONLY
Open

Add textOnly paramater to get_links_from_email#12
koshie wants to merge 1 commit into
lovelysystems:masterfrom
koshie:GET_TEXT_ONLY

Conversation

@koshie

@koshie koshie commented Sep 27, 2013

Copy link
Copy Markdown

Useful when mail is format in text only

Useful when mail is format in text only
@koshie

koshie commented Oct 10, 2013

Copy link
Copy Markdown
Author

I need this option because sometimes I need to bring back a link in format text and not in format HTML. It can be probably useful for someone else.

@nickmayorga

Copy link
Copy Markdown

I second this, as I was trying to use get_email_body since the email body was 'text/plain; charset=UTF-8', and get_links_from_email would only search for an anchor tag.

get_email_body is also removing URL parameters since the '=' sign is getting filtered out as quoted-printable data. as an example I have a URL with a token parameter like 'change?token=1023gf5m5lff...', where the first '=10' is getting erased from the rest of the email body that is returned. It looks like someone tried to correct this with a decode('utf-8'), but since it is already going through the quoted-printable filter first, I don't that wouldn't help.

@schwendinger

Copy link
Copy Markdown
Member

Deprecation Warning
+++++++++++++++++++

Lovely Systems does not not support this package anymore and
do not have any follow up package in the same area. If anyone is
interested to continue our efforts and would like to
manage the contributors in this open source project,
feel free to fork the package and give me a hint, so I can
create a link to your fork!

best regards, Manfred (Github: schwendinger, schwendinger at lovelysystems.com)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants