pecan is missing the de-dup and empty field stripping logic of the legacy controller

Bug #1714384 reported by Kevin Benton
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
High
Kevin Benton

Bug Description

When a user passes in duplicate fields and empty fields, the old API controller would strip these out before passing them to the plugin.

Pecan should do the same thing to preserve parity with the old controller in case plugins are sensitive to these invalid filters.

Changed in neutron:
importance: Undecided → High
Changed in neutron:
assignee: nobody → Kevin Benton (kevinbenton)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (master)

Reviewed: https://review.openstack.org/499431
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=700d609ace818457093dc653264f84ffbdae4d86
Submitter: Jenkins
Branch: master

commit 700d609ace818457093dc653264f84ffbdae4d86
Author: Kevin Benton <email address hidden>
Date: Wed Aug 30 20:14:20 2017 -0700

    Pecan: strip duplicate and empty user fields

    Remove duplicated and empty fields from users requests
    in Pecan to preserve the old legacy API controller behavior.

    Closes-Bug: #1714384
    Change-Id: I1afc24b146a8fcc6c8ebae708f32dd7c1795292e

Changed in neutron:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (stable/pike)

Fix proposed to branch: stable/pike
Review: https://review.openstack.org/502978

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (stable/pike)

Reviewed: https://review.openstack.org/502978
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=6fe51999d53c45d04fdbe8a5b3878c3d267a7cbe
Submitter: Jenkins
Branch: stable/pike

commit 6fe51999d53c45d04fdbe8a5b3878c3d267a7cbe
Author: Kevin Benton <email address hidden>
Date: Wed Aug 30 20:14:20 2017 -0700

    Pecan: strip duplicate and empty user fields

    Remove duplicated and empty fields from users requests
    in Pecan to preserve the old legacy API controller behavior.

    Closes-Bug: #1714384
    Change-Id: I1afc24b146a8fcc6c8ebae708f32dd7c1795292e
    (cherry picked from commit 700d609ace818457093dc653264f84ffbdae4d86)

tags: added: in-stable-pike
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/neutron 11.0.1

This issue was fixed in the openstack/neutron 11.0.1 release.

tags: added: neutron-proactive-backport-potential
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/neutron 12.0.0.0b1

This issue was fixed in the openstack/neutron 12.0.0.0b1 development milestone.

tags: removed: neutron-proactive-backport-potential
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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