From 357a41c4281181cd0cffa04b377cdfc3020eaf1f Mon Sep 17 00:00:00 2001 From: zhouhenglc Date: Fri, 20 Sep 2019 14:46:36 +0800 Subject: [PATCH] fix get the name servers for a zone not validate zone_id now the api of get /v2/zones/{zone_id}/nameservers not validate zone_id, if set zone_id contain spaces, will trigger DBError and return 500. Change-Id: Ia3e07c42dc548207aa7ff9efcc35a215c9660e54 (cherry picked from commit 0ba52bfb62cb64992a06b64c44f3e6a9dcadc193) --- designate/api/v2/controllers/zones/nameservers.py | 2 ++ 1 file changed, 2 insertions(+) diff --git a/designate/api/v2/controllers/zones/nameservers.py b/designate/api/v2/controllers/zones/nameservers.py index 9d4fce63b..8f5c7e263 100644 --- a/designate/api/v2/controllers/zones/nameservers.py +++ b/designate/api/v2/controllers/zones/nameservers.py @@ -16,6 +16,7 @@ import pecan from oslo_log import log as logging +from designate import utils from designate.api.v2.controllers import rest from designate.objects.adapters import DesignateAdapter @@ -25,6 +26,7 @@ LOG = logging.getLogger(__name__) class NameServersController(rest.RestController): @pecan.expose(template='json:', content_type='application/json') + @utils.validate_uuid("zone_id") def get_all(self, zone_id): """List NameServers for Zone""" request = pecan.request