oslo.messaging

Make oslo rpc sanitization configuration project specific

Bug #1095587 reported by Michael Still
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
oslo.messaging
Triaged
Medium
Unassigned

Bug Description

Oslo incubator contains code to sanitize RPCs, but it is specific to nova. We should instead let each project configure it when they load the rpc libraries.

Revision history for this message
Mark McLoughlin (markmc) wrote :

You can see the nova specific code here:

https://github.com/openstack/oslo-incubator/blob/15ae704d9/openstack/common/rpc/common.py#L166

    SANITIZE = {'set_admin_password': ('new_pass',),
                'run_instance': ('admin_password',), }

I could imagine Nova doing something like this:

  rpc.register_sanitize_methods(SANITIZE)

Changed in oslo:
milestone: none → grizzly-3
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to oslo-incubator (master)

Fix proposed to branch: master
Review: https://review.openstack.org/19604

Changed in oslo:
status: Triaged → In Progress
Mark McLoughlin (markmc)
Changed in oslo:
milestone: grizzly-3 → none
Revision history for this message
Mark McLoughlin (markmc) wrote :

Patch was abandoned 6 weeks ago or so

Changed in oslo:
assignee: Michael Still (mikalstill) → nobody
status: In Progress → Triaged
Doug Hellmann (doug-hellmann)
affects: oslo-incubator → oslo.messaging
Revision history for this message
Mehdi Abaakouk (sileht) wrote :

fix_password is now part of oslo.utils, oslo.messaging just use it, all of this cases are already handled by oslo.utils.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.