openstack/keystone
Code Issues Proposed changes

Fix host:port handling

Browse Source 
When we check the EC2 signature without the port part of the host value
received, we should properly split host:port. Keep in mind the splitting
should work for values like [fc00::]:123 too.

Change-Id: I1d90dfcea3568e2a9b22069daa428ea6a2a38bd6
Closes-Bug: #1988168
This commit is contained in:
Bence Romsics 2022-08-29 16:03:44 +02:00
parent 051aca8e8a
commit 6c35b366e3
1 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
keystone/api/ec2tokens.py
View File

@ -12,6 +12,8 @@
# This file handles all flask-restful resources for /v3/ec2tokens
import urllib.parse
import flask
import http.client
from keystoneclient.contrib.ec2 import utils as ec2_utils
@ -42,8 +44,8 @@ class EC2TokensResource(EC2_S3_Resource.ResourceBase):
# NOTE(vish): Some client libraries don't use the port when
# signing requests, so try again without the port.
elif ':' in credentials['host']:
hostname, _port = credentials.split(':')
credentials['host'] = hostname
parsed = urllib.parse.urlsplit('//' + credentials['host'])
credentials['host'] = parsed.hostname
# NOTE(davechen): we need to reinitialize 'signer' to avoid
# contaminated status of signature, this is similar with
# other programming language libraries, JAVA for example.