openstack
/
heat-cfnclient
Code Issues Proposed changes

Fix the standard port that is inferred from url scheme 

This is a port of this review (https://review.openstack.org/#/c/39210/2)
to try and cleanup and old bug.

Change-Id: Idf93053c9f4a49f65d9d52e879a88d79c9fd3170
Co-Authored-by: Martin H Berwanger <martin.berwanger@gmail.com>
Fixes: bug #1205317
This commit is contained in:
Angus Salkeld 2014-12-02 15:29:31 +10:00
parent f15de52075
commit a010238c54
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
heat_cfnclient/common/client.py
View File

@ -312,7 +312,8 @@ class BaseClient(object):
self.use_ssl = parsed.scheme == 'https'
if self.host is None:
self.host = parsed.hostname
self.port = parsed.port or 80
std_port = 443 if self.use_ssl else self.DEFAULT_PORT
self.port = parsed.port or std_port
self.doc_root = parsed.path
# ensure connection kwargs are re-evaluated after the service catalog