openstack/oslo.messaging
Code Issues Proposed changes

Use the hostname from the Transport for GSSAPI Authentication

Browse Source 
Previously, the AMQP 1.0 driver did not pass the proper hostname to
pyngus.  This prevented GSSAPI authentication from working properly.

Change-Id: Ibc6678e7cbae6dd5108d1650dbb8ddf837aa3648
Closes-Bug: #1503258
This commit is contained in:
Kenneth Giusti 2015-10-01 16:43:56 -04:00
parent 7eadea5bec
commit 1b605c1466
2 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
oslo_messaging/_drivers/protocols/amqp/controller.py
View File

@ -460,7 +460,7 @@ class Controller(pyngus.ConnectionEventHandler):
def _do_connect(self): def _do_connect(self):
"""Establish connection and reply subscription on processor thread.""" """Establish connection and reply subscription on processor thread."""
host = self.hosts.current host = self.hosts.current
conn_props = {} conn_props = {'hostname': host.hostname}
if self.idle_timeout: if self.idle_timeout:
conn_props["idle-time-out"] = float(self.idle_timeout) conn_props["idle-time-out"] = float(self.idle_timeout)
if self.trace_protocol: if self.trace_protocol:

1
tox.ini
View File

@ -5,6 +5,7 @@ envlist = py34,py27,pep8
setenv = setenv =
VIRTUAL_ENV={envdir} VIRTUAL_ENV={envdir}
OS_TEST_TIMEOUT=30 OS_TEST_TIMEOUT=30
PN_TRACE_FRM=1
deps = -r{toxinidir}/requirements.txt deps = -r{toxinidir}/requirements.txt
-r{toxinidir}/test-requirements.txt -r{toxinidir}/test-requirements.txt
commands = python setup.py testr --slowest --testr-args='{posargs}' commands = python setup.py testr --slowest --testr-args='{posargs}'