Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unable to remove the part containing '--- Forwarded message ---' #220

Open
chenxg283 opened this issue Jun 30, 2021 · 2 comments
Open

Unable to remove the part containing '--- Forwarded message ---' #220

chenxg283 opened this issue Jun 30, 2021 · 2 comments

Comments

@chenxg283
Copy link

hi, if my email content is like below:

"""reply

----- Forwarded message ----

some forwarded text"""

the markers will be "tefet".

However, in the function "process_marked_lines" in quotations.py, there is this code:
if re.match('[te]*f', markers):
return_flags[:] = [False, -1, -1]
return lines

This will return the whole email content as the reply without removing the "---- Forwarded message ----" and subsequent text.

Is this intended?

@atymic
Copy link

atymic commented Jul 13, 2021

Yea, noticed the same issue.

chenxg283 added a commit to chenxg283/talon that referenced this issue Jul 15, 2021
Fix issue mailgun#220
cut out text lines starting from " -- forwarded messages --"
@chenxg283
Copy link
Author

I have created a PR to fix this issue. If anyone has a better solution, please share as well. thanks.

circlingthesun pushed a commit to circlingthesun/talon that referenced this issue May 25, 2023
Fix issue mailgun#220
cut out text lines starting from " -- forwarded messages --"
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants