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

0.9.9 drops carriage returns #616

Closed
juancabrio opened this issue Jul 1, 2022 · 7 comments
Closed

0.9.9 drops carriage returns #616

juancabrio opened this issue Jul 1, 2022 · 7 comments

Comments

@juancabrio
Copy link

Hi,

I cat a text file into apprise as the body of an email. This works great with 0.9.8 and below. When I use 0.9.9 the returns in my text file all get stripped and the email body is one long line. Looking for a way forward with 0.9.9 but happy to stay with 0.9.8 if needed.

Cheers

Juan

@caronc
Copy link
Owner

caronc commented Jul 1, 2022

Interesting, so you're just using the apprise CLI? Or are you using the Apprise API?

@juancabrio
Copy link
Author

The CLI. Like this.
cat file | /usr/local/bin/apprise -t "test email" mailto://bla@bla
Happy to be corrected if this is not the preferred method.
Cheers

@caronc
Copy link
Owner

caronc commented Jul 1, 2022

No, what you're doing is great. I'll try to have a look at this over the weekend. Something is off based on what you're describing. I don't recall touching email in the last release. 🤔

I'll keep you posted. In the meantime (doubtfully related; but just in case) what mail service are you using? Is it a private or public one (like Hotmail or Gmail)?

@juancabrio
Copy link
Author

I've tried both gmail and outlook with the same result.
I can easily test back and forth using pip to install the different versions to reproduce.
Let me know if you want me to dump a requirements.txt form virtual env. If that helps.

@juancabrio
Copy link
Author

Actually, my previous comment isn't entirely accurate. I relay through a local SMTP server.

@caronc
Copy link
Owner

caronc commented Jul 1, 2022

I think I see where i broke the code in the last release. It was to accommodate a Telegram issue. It looks like i broke the very feature you're inquiring about here. 🫣

Shouldn't be too much of an issue to fix. Thank you for reporting this

@caronc
Copy link
Owner

caronc commented Jul 1, 2022

If you access the main branch of apprise, the issue should be gone now for you. There was also a bad unit test i corrected so that this won't break again in future versions.

@caronc caronc closed this as completed Jul 1, 2022
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