Fix moto_server handling of unsigned requests
Certain AWS requests are unsigned. Moto in standalone server mode implements an heuristic to deduce the endpoint and region based on the X-Amz-Target HTTP header. This commit extends this concept to add additional endpoints that used unsigned requests at times.
This commit is contained in:
parent
7c17fcd21d
commit
19fef76a5f
@ -21,6 +21,16 @@ from moto.core.utils import convert_flask_to_httpretty_response
|
|||||||
HTTP_METHODS = ["GET", "POST", "PUT", "DELETE", "HEAD", "PATCH"]
|
HTTP_METHODS = ["GET", "POST", "PUT", "DELETE", "HEAD", "PATCH"]
|
||||||
|
|
||||||
|
|
||||||
|
DEFAULT_SERVICE_REGION = ('s3', 'us-east-1')
|
||||||
|
|
||||||
|
# Map of unsigned calls to service-region as per AWS API docs
|
||||||
|
# https://docs.aws.amazon.com/cognito/latest/developerguide/resource-permissions.html#amazon-cognito-signed-versus-unsigned-apis
|
||||||
|
UNSIGNED_REQUESTS = {
|
||||||
|
'AWSCognitoIdentityService': ('cognito-identity', 'us-east-1'),
|
||||||
|
'AWSCognitoIdentityProviderService': ('cognito-idp', 'us-east-1'),
|
||||||
|
}
|
||||||
|
|
||||||
|
|
||||||
class DomainDispatcherApplication(object):
|
class DomainDispatcherApplication(object):
|
||||||
"""
|
"""
|
||||||
Dispatch requests to different applications based on the "Host:" header
|
Dispatch requests to different applications based on the "Host:" header
|
||||||
@ -50,6 +60,32 @@ class DomainDispatcherApplication(object):
|
|||||||
|
|
||||||
raise RuntimeError('Invalid host: "%s"' % host)
|
raise RuntimeError('Invalid host: "%s"' % host)
|
||||||
|
|
||||||
|
def infer_service_region(self, environ):
|
||||||
|
auth = environ.get('HTTP_AUTHORIZATION')
|
||||||
|
if auth:
|
||||||
|
# Signed request
|
||||||
|
# Parse auth header to find service assuming a SigV4 request
|
||||||
|
# https://docs.aws.amazon.com/general/latest/gr/sigv4-signed-request-examples.html
|
||||||
|
# ['Credential=sdffdsa', '20170220', 'us-east-1', 'sns', 'aws4_request']
|
||||||
|
try:
|
||||||
|
credential_scope = auth.split(",")[0].split()[1]
|
||||||
|
_, _, region, service, _ = credential_scope.split("/")
|
||||||
|
return service, region
|
||||||
|
except ValueError:
|
||||||
|
# Signature format does not match, this is exceptional and we can't
|
||||||
|
# infer a service-region. A reduced set of services still use
|
||||||
|
# the deprecated SigV2, ergo prefer S3 as most likely default.
|
||||||
|
# https://docs.aws.amazon.com/general/latest/gr/signature-version-2.html
|
||||||
|
return DEFAULT_SERVICE_REGION
|
||||||
|
else:
|
||||||
|
# Unsigned request
|
||||||
|
target = environ.get('HTTP_X_AMZ_TARGET')
|
||||||
|
if target:
|
||||||
|
service, _ = target.split('.', 1)
|
||||||
|
return UNSIGNED_REQUESTS.get(service, DEFAULT_SERVICE_REGION)
|
||||||
|
# S3 is the last resort when the target is also unknown
|
||||||
|
return DEFAULT_SERVICE_REGION
|
||||||
|
|
||||||
def get_application(self, environ):
|
def get_application(self, environ):
|
||||||
path_info = environ.get('PATH_INFO', '')
|
path_info = environ.get('PATH_INFO', '')
|
||||||
|
|
||||||
@ -66,19 +102,7 @@ class DomainDispatcherApplication(object):
|
|||||||
else:
|
else:
|
||||||
host = environ['HTTP_HOST'].split(':')[0]
|
host = environ['HTTP_HOST'].split(':')[0]
|
||||||
if host in {'localhost', 'motoserver'} or host.startswith("192.168."):
|
if host in {'localhost', 'motoserver'} or host.startswith("192.168."):
|
||||||
# Fall back to parsing auth header to find service
|
service, region = self.infer_service_region(environ)
|
||||||
# ['Credential=sdffdsa', '20170220', 'us-east-1', 'sns', 'aws4_request']
|
|
||||||
try:
|
|
||||||
_, _, region, service, _ = environ['HTTP_AUTHORIZATION'].split(",")[0].split()[
|
|
||||||
1].split("/")
|
|
||||||
except (KeyError, ValueError):
|
|
||||||
# Some cognito-idp endpoints (e.g. change password) do not receive an auth header.
|
|
||||||
if environ.get('HTTP_X_AMZ_TARGET', '').startswith('AWSCognitoIdentityProviderService'):
|
|
||||||
service = 'cognito-idp'
|
|
||||||
else:
|
|
||||||
service = 's3'
|
|
||||||
|
|
||||||
region = 'us-east-1'
|
|
||||||
if service == 'dynamodb':
|
if service == 'dynamodb':
|
||||||
if environ['HTTP_X_AMZ_TARGET'].startswith('DynamoDBStreams'):
|
if environ['HTTP_X_AMZ_TARGET'].startswith('DynamoDBStreams'):
|
||||||
host = 'dynamodbstreams'
|
host = 'dynamodbstreams'
|
||||||
|
Loading…
x
Reference in New Issue
Block a user