search

lavab / web

AngularJS web client of Lavaboom's email service
https://mail.lavaboom.com
38 stars 21 forks source link

Empty email! #817

Closed flixi closed 9 years ago

flixi commented 9 years ago

https://gist.github.com/flixi/e2f53bcfb1f2d88753f9 bildschirmfoto 2015-06-02 um 20 15 25 Emails from Twitter don'T work, and from DHL neither, the same issue I suppose.

vviikk commented 9 years ago

I'm also getting this... all text seem to be stripped. here's the sent email in gmail image

Here's the received email in lavaboom: image

vviikk commented 9 years ago

ok this is even more f'd up. I replied to the previous email (the image was there, but no text) and i received this in gmail: image

let4be commented 9 years ago

I just checked and indeed - only picture shows and no text. The thing is email came without text from API, most likely Mailer somehow damaged it before encoding and giving via API see: https://github.com/lavab/mailer/issues/51

let4be commented 9 years ago

Actually it seems it was mistake in web... Will post more details soon

let4be commented 9 years ago

I've fixed very similar bug in web but those issues are confirmed to be different, @pzduniak asap

let4be commented 9 years ago

Need this to be worked on: https://github.com/lavab/mailer/issues/51

joergG commented 9 years ago

@let4be - can we close this issue? since we have a separate bug in lavab/mailer#51?

let4be commented 9 years ago

sure, this is caused by lavab/mailer#51 and appear only on unencrypted emails most likely mailer has issues with parsing

There were some debates about switching to front-end email parsing - exactly the same we do for pgp/mine now, but looks like we postponed this idea.

It could simplify whole thing and help to get rid of some bugs

@andreis what's your input on this because right now we have front-end mail parsing + back-end mail parsing on mailer's side

joergG commented 9 years ago

so FE-mail parsing would bring huge benefits? how much effort would that be? (just to get a feeling)