Commit Graph

406 Commits

Author SHA1 Message Date
minglis
9b99782662 Merge pull request #83 from alphagov/celery-send-email-code
Celery send email code
2016-02-18 11:03:23 +00:00
Rebecca Law
4d9d511575 Merge pull request #81 from alphagov/celery-send-sms-code
Use Celery to send sms code
2016-02-18 10:51:16 +00:00
Rebecca Law
aff948e9ca Merge branch 'celery-send-sms-code' into celery-send-email-code
Conflicts:
	app/celery/tasks.py
	app/user/rest.py
	scripts/run_celery.sh
2016-02-18 10:50:14 +00:00
Rebecca Law
b286b4356c Update queue name with dashes 2016-02-18 10:45:25 +00:00
Rebecca Law
3ce6758f3e Change names for clarity sake.
Rename test.
What we do when the validation_code fails to send is still to be decided.
2016-02-18 09:59:18 +00:00
Rebecca Law
c1df61a7d4 Use celery to send the email verification code. 2016-02-18 09:52:27 +00:00
Rebecca Law
9073814d9f I have an issue with the test, not sure why? 2016-02-17 17:48:23 +00:00
NIcholas Staples
13b7a89c8c Merge pull request #82 from alphagov/db-error-handling
Wire up error handlers.
2016-02-17 17:23:27 +00:00
Adam Shimali
4f33b6f406 Wire up error handlers.
Replace some 400s with more appropriate 500s.

DAO methods that cause unexpected exceptions get caught and
logged by errors.py 500 error handler.
2016-02-17 17:04:50 +00:00
Rebecca Law
66cf6cfd30 Use Firetext client 2016-02-17 15:52:09 +00:00
Rebecca Law
7afa87c367 Merge branch 'master' into celery-send-sms-code
Conflicts:
	tests/app/celery/test_tasks.py
2016-02-17 15:43:57 +00:00
Rebecca Law
d022d036dc Celery task to send the sms verify code.
Each celery task will use it's own queue.
2016-02-17 15:41:33 +00:00
NIcholas Staples
b98d8b9996 Merge pull request #80 from alphagov/fire-text-client
Fire text client
2016-02-17 14:55:02 +00:00
Martyn Inglis
53bdedf8f3 Test properties for firetext 2016-02-17 14:27:28 +00:00
Martyn Inglis
36eee56f17 Adding boto for python2 as Celery seems to need it. 2016-02-17 14:03:40 +00:00
Martyn Inglis
837c9b7cdb Removed logging, and make fire text only client. 2016-02-17 13:59:01 +00:00
Rebecca Law
c7cebc4d00 Merge pull request #78 from alphagov/move-sms-notifications-into-celery
Move sms notifications into celery
2016-02-17 13:29:45 +00:00
Martyn Inglis
226459132a Basic (Very basic) implementation of the fire text API.
[https://www.firetext.co.uk/docs#sendingsms](https://www.firetext.co.uk/docs#sendingsms)

Not to be merged. This API has a limit on it at the moment that will need to be removed before it is used in anger.
2016-02-17 12:57:51 +00:00
Rebecca Law
d6bea65626 Merge branch 'move-sms-notifications-into-celery' into celery-send-sms-code 2016-02-17 11:50:36 +00:00
Martyn Inglis
58ae041b64 AWS scripts for celery 2016-02-17 11:36:14 +00:00
Martyn Inglis
e59e66fa21 Run celery needed to read in secrets 2016-02-17 11:25:57 +00:00
Martyn Inglis
cebca61f8e Travis deploy group for delivery version of api 2016-02-17 11:21:49 +00:00
Martyn Inglis
95cdfe08d1 Updated logging in run-celery script to log to application logs
- poll every second
- visibility of 1 minute
2016-02-17 11:18:27 +00:00
Rebecca Law
5f4b6a7db9 Fix db.py script for credstash secrets 2016-02-17 11:01:15 +00:00
NIcholas Staples
ab1279116f Merge pull request #77 from alphagov/clarify-readme
Updated sample values for secret keys in environment.sh to match
2016-02-17 10:29:44 +00:00
Martyn Inglis
e0e47b40fc Setup celery config 2016-02-17 10:22:25 +00:00
Adam Shimali
914b650a46 Updated sample values for secret keys in environment.sh to match
that suggested in delivery app.

Called out the reason for unique prefixes or queue names.
2016-02-17 10:20:40 +00:00
Chris Hill-Scott
bb88fd79dc Merge pull request #76 from alphagov/update-readme
Update README with set up instructions.
2016-02-17 09:58:21 +00:00
Rebecca Law
8b4641021a Update README with set up instructions. 2016-02-17 09:49:36 +00:00
Martyn Inglis
85a8e6d2b4 Added moto back for methods not yet celery'd 2016-02-17 09:47:34 +00:00
Martyn Inglis
ec8955503f Twilio params into setup file for tests 2016-02-17 09:34:57 +00:00
Martyn Inglis
73d0e67e68 Removed alpha api client 2016-02-17 09:22:06 +00:00
Martyn Inglis
cb265896a7 Merge branch 'master' into move-sms-notifications-into-celery
Conflicts:
	app/__init__.py
	tests/conftest.py
2016-02-17 09:14:37 +00:00
Martyn Inglis
1311af5920 Twilio client takes a 'to' number not a notification 2016-02-17 09:12:13 +00:00
Martyn Inglis
a2341be0e2 Don't send the SMS if we have failed to save in the database 2016-02-16 17:42:04 +00:00
Martyn Inglis
0933e5c647 Building tests for the tasks class 2016-02-16 17:17:02 +00:00
minglis
c1e3317967 Merge pull request #75 from alphagov/update-config
Update config
2016-02-16 16:39:41 +00:00
Rebecca Law
af86520e95 Add verify_code_email_address 2016-02-16 16:03:30 +00:00
Rebecca Law
08ba5de61b Update config
Source the configuration from an environment file, this way it is similar to how the aws environment works
2016-02-16 16:01:50 +00:00
Martyn Inglis
18d63e241b Extracted serialiser for encryption into a flask module
- allows mocking easier
- shared across methods
- not built everytime
2016-02-16 15:28:30 +00:00
Martyn Inglis
e42da7dd54 Fixing up tests to validate the call to the celery tasks.
- mocker used to test call or otherwise of the task
- no new tests just a spring clean
2016-02-16 14:06:56 +00:00
Chris Hill-Scott
7ff5f6c45a Merge pull request #74 from alphagov/update_from_email_address
Update to config to pull out from email address.
2016-02-16 11:56:56 +00:00
Martyn Inglis
655beddba6 Fixed up the get_notitication endpoint
- returns a notification
2016-02-16 11:22:44 +00:00
Nicholas Staples
20650a91e0 Update to config to pull out from email address. 2016-02-16 11:10:02 +00:00
Martyn Inglis
223cb8c2dd Made SMS messages go through celery
- twilio client pulled in from delivery app
- made method to perform task
2016-02-15 16:01:14 +00:00
NIcholas Staples
025a6fb47e Merge pull request #72 from alphagov/fix-template-validation
Fix validation of template for service.
2016-02-15 15:10:16 +00:00
Rebecca Law
fcaed04918 Fix validation of template for service.
When the delivery app creates the sms on behalf of the service,
the validation was failing.
2016-02-15 15:02:19 +00:00
Martyn Inglis
ffbe94f390 Merge branch 'master' into celery-spike
Conflicts:
	app/notifications/rest.py
2016-02-15 11:12:07 +00:00
Martyn Inglis
16c5e7bf10 Celery task added 2016-02-15 11:11:20 +00:00
Chris Hill-Scott
e7ef5945e5 Merge pull request #71 from alphagov/validate_template_for_a_service
Validate template for a service
2016-02-12 15:14:24 +00:00