1128553 - Use the smtp.mozilla.com host for sending e-mails

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::UI Tests, defect)

Description

[Dave Hunt [:davehunt] [he/him] ⌚BST]

It appears that a couple of tests send e-mails via SMTP to check they are received in the e-mail app. These are currently set up to use smtp.gmail.com, but we would need to open a firewall route to allow this from our datacenter. It would be better if we can instead use smtp.mozilla.com.

I recommend a new 'smtp' key in the 'email' test variables, which will be used when we send e-mails from Python rather than from the devices themselves. Something like:

"email" {
  "smtp": {
    "email": "[e-mail]",
    "hostname": "smtp.mozilla.com",
    "port": 25
  }
}

I also noticed that EmailUtil assumes a password will be present, however this should be conditional. When sending e-mail from our datacenter there's no requirement for authentication.

Comment 1

[Florin Strugariu [:Bebe]]

Attachment: Link to Github pull-request: https://github.com/mozilla/webqa-credentials/pull/190

Comment 2

[GitHub Autolander Bot]

Attachment: [PullReq] bebef1987:email to mozilla-b2g:master

Comment 3

[Dave Hunt [:davehunt] [he/him] ⌚BST]

Bebe: The patches look good, but the commit messages do not reflect the changes. Please update these so I can merge them.

Comment 4

[Florin Strugariu [:Bebe]]

Done

Dave I think this can be merged

Comment 5

[Dave Hunt [:davehunt] [he/him] ⌚BST]

Comment on attachment 8559881 [details] [review]
Link to Github pull-request: https://github.com/mozilla/webqa-credentials/pull/190

I tweaked the commit message on landing: https://github.com/mozilla/webqa-credentials/commit/9b1fd83b5c7153a4a5503d5a4d98b47e68167543

Once we're confident our nodes have pulled the latest credentials files we can merge the gaia patch.

Comment 6

[Dave Hunt [:davehunt] [he/him] ⌚BST]

Landed in:
https://github.com/mozilla-b2g/gaia/commit/4c0805f9b7a9d5fdcf7dee2658d98e0b1f16033e