Mitch Garnaat

Merge pull request #33 from ryansb/develop

`update_event_sources` fails on SNS and S3 event sources
......@@ -132,18 +132,26 @@ class S3EventSource(EventSource):
def add(self, function):
notification_spec = {
'CloudFunctionConfiguration': {
'LambdaFunctionConfigurations':[
{
'Id': self._make_notification_id(function.name),
'Events': [e for e in self._config['events']],
'CloudFunction': function.arn}}
'LambdaFunctionArn': function.arn,
}
]
}
try:
response = self._s3.put_bucket_notification(
response = self._s3.put_bucket_notification_configuration(
Bucket=self._get_bucket_name(),
NotificationConfiguration=notification_spec)
LOG.debug(response)
except Exception:
except Exception as exc:
LOG.debug(exc.response)
LOG.exception('Unable to add S3 event source')
def update(self, function):
self.add(function)
def remove(self, function):
LOG.debug('removing s3 notification')
response = self._s3.get_bucket_notification(
......@@ -200,6 +208,9 @@ class SNSEventSource(EventSource):
except Exception:
LOG.exception('Unable to add SNS event source')
def update(self, function):
self.add(function)
def remove(self, function):
LOG.debug('removing SNS event source')
try:
......