all_tenants filter is currently broken
Bug #1481309 reported by
Endre Karlson
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 |
Changed in designate: | |
status: | New → Triaged |
Changed in designate: | |
status: | Triaged → In Progress |
Changed in designate: | |
status: | Fix Committed → Fix Released |
Changed in designate: | |
milestone: | liberty-3 → 1.0.0 |
To post a comment you must log in.
Reviewed: https:/ /review. openstack. org/209445 /git.openstack. org/cgit/ openstack/ designate/ commit/ ?id=e161fbb80cb 2355a7d89bc7e19 7e7648e274ee37
Committed: https:/
Submitter: Jenkins
Branch: master
commit e161fbb80cb2355 a7d89bc7e197e76 48e274ee37
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: I81fa781c747531 ae89bb6b51e4e61 07add45dd9a