SomeAverageDev / cozy-downloader

5 stars 0 forks source link

Email sent from cozy@localhost instead of cozy@mydomain.tld #13

Closed ZeHiro closed 8 years ago

ZeHiro commented 8 years ago

even if my email is set. Somethins it wrong here: https://github.com/SomeAverageDev/cozy-downloader/blob/master/server/controllers/download.js#L297

Shouldn't is be : user.email || 'cozy@localhost' + '>'

It results in : Diagnostic-Code: smtp; 504 5.5.2 <cozy@localhost>: Sender address rejected: need fully-qualified address From my email provider (Gandi)

SomeAverageDev commented 8 years ago

that's right, there is a mistake here, I will fix it asap

it should probably be something like (user.email || 'cozy@localhost' )+'>'

thanks

SomeAverageDev commented 8 years ago

fixed in committed version 0.0.18, @ZeHiro could you try it on your side ?

ZeHiro commented 8 years ago

Works fine :+1: