Wrong property to look up remote address
Bug #1967683 reported by
Takashi Kajinami
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Cinder |
Fix Released
|
Low
|
Takashi Kajinami | ||
OpenStack Compute (nova) |
Fix Released
|
Undecided
|
Takashi Kajinami | ||
OpenStack Shared File Systems Service (Manila) |
Fix Released
|
Undecided
|
Takashi Kajinami | ||
ec2-api |
Fix Released
|
Low
|
Takashi Kajinami |
Bug Description
Currently, remote_address attribute of the Reqeust object is used to look up client address in multiple places.
~~~
def _set_request_
"""Sets request context based on parameters and request."""
remote_address = getattr(req, 'remote_address', '127.0.0.1')
~~~
However, webob.Request has no remote_address attribute but only remote_addr attribute.
https:/
no longer affects: | masakari |
Changed in cinder: | |
assignee: | nobody → Takashi Kajinami (kajinamit) |
Changed in nova: | |
assignee: | nobody → Takashi Kajinami (kajinamit) |
Changed in manila: | |
assignee: | nobody → Takashi Kajinami (kajinamit) |
Changed in ec2-api: | |
assignee: | nobody → Takashi Kajinami (kajinamit) |
Changed in ec2-api: | |
importance: | Undecided → Low |
Changed in cinder: | |
importance: | Undecided → Low |
To post a comment you must log in.
Fix proposed to branch: master /review. opendev. org/c/openstack /cinder/ +/836247
Review: https:/