{"push_id":2558818293,"size":1,"distinct_size":1,"ref":"refs/heads/pb/3","head":"da418066b8acd9e3c842817f7ca2f9fbd3e5565d","before":"1661ac18f43c77498a69f3731f26671802ac9aca","commits":[{"sha":"da418066b8acd9e3c842817f7ca2f9fbd3e5565d","author":{"name":"Phil Booth","email":"f8d194a6b22723e89c282a4347656d46cf7386a9@gmail.com"},"message":"feat(db): check the emailBounces table before sending email\n\nLoads bounce limits from config and then checks email addresses against\nthose limits before sending the email. If any address violates a limit,\nthe request will fail with status 429 and email will not be sent.\n\nAs it introduces a dependency on the fxa-auth-db-mysql repo, some extra\nscripting has been thrown in to make working with node.js easier.\n\nRunning `./tdb` will check whether an `fxa-auth-db-mysql` directory\nexists and, if not, clone the repo. Then it will check for running db\nserver processes and, if none are found, run `npm i`, run the db patcher\nand start the db server in a background process. Then it finishes by\ninvoking the existing `./t` script.\n\nThis is not ideal because it leaves the node process running in the\nbackground after tests have finished. But I figure it's good enough for\nnow because it only leaves one of them running, they don't multiply with\nsubsequent test runs. And you don't have to use the new script of\ncourse, if you already have the db server running in fxa-local-dev, you\ncan just run `./t` and the tests should pass. The new script is mostly\nthere for Travis' benefit.\n\nLonger term, if this service is going to stick around, we can do\nsomething more permanent to manage it properly.","distinct":true,"url":"https://api.github.com/repos/mozilla/fxa-email-service/commits/da418066b8acd9e3c842817f7ca2f9fbd3e5565d"}]}
{"actor":{"display_login":"philbooth"}}