Merge pull request #1666 from ogckw/master

Change SNS HTTP headers to actual setting
This commit is contained in:
Steve Pulec 2018-06-03 20:25:29 -04:00 committed by GitHub
commit 7951eeddc3
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 2 additions and 2 deletions

View File

@ -101,7 +101,7 @@ class Subscription(BaseModel):
sqs_backends[region].send_message(queue_name, enveloped_message)
elif self.protocol in ['http', 'https']:
post_data = self.get_post_data(message, message_id, subject)
requests.post(self.endpoint, json=post_data)
requests.post(self.endpoint, json=post_data, headers={'Content-Type': 'text/plain; charset=UTF-8'})
elif self.protocol == 'lambda':
# TODO: support bad function name
# http://docs.aws.amazon.com/general/latest/gr/aws-arns-and-namespaces.html

View File

@ -253,7 +253,7 @@ def test_publish_to_sqs_in_different_region():
@mock_sns
def test_publish_to_http():
def callback(request):
request.headers["Content-Type"].should.equal("application/json")
request.headers["Content-Type"].should.equal("text/plain; charset=UTF-8")
json.loads.when.called_with(
request.body.decode()
).should_not.throw(Exception)