all_tenants filter is currently broken

Bug #1481309 reported by Endre Karlson
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Designate
Fix Released
High
Endre Karlson

Bug Description

all_tenants filter is disallowed after the filters enforcement patch landed.

We should update the KS middleware to filter out this query param and anything alike it.

Changed in designate:
importance: Undecided → High
assignee: nobody → Endre Karlson (endre-karlson)
milestone: none → liberty-3
Kiall Mac Innes (kiall)
Changed in designate:
status: New → Triaged
Changed in designate:
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to designate (master)

Reviewed: https://review.openstack.org/209445
Committed: https://git.openstack.org/cgit/openstack/designate/commit/?id=e161fbb80cb2355a7d89bc7e197e7648e274ee37
Submitter: Jenkins
Branch: master

commit e161fbb80cb2355a7d89bc7e197e7648e274ee37
Author: Endre Karlson <email address hidden>
Date: Wed Aug 5 12:23:54 2015 +0200

    Remove auth* related param keys in middleware

    We need to remove parameters that are used for authentication if not you will
    get caught in the filter check in the API.

    Added tests for this also and split up make_context into smaller methods.

    Closes-Bug: 1481309

    Change-Id: I81fa781c747531ae89bb6b51e4e6107add45dd9a

Changed in designate:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in designate:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in designate:
milestone: liberty-3 → 1.0.0
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.