From 38aeed4f2c629ba79a7c176a74cdba3e0e0573dd Mon Sep 17 00:00:00 2001
From: Jimmy Zelinskie <jimmyzelinskie@gmail.com>
Date: Fri, 29 Jan 2016 16:43:17 -0500
Subject: [PATCH] api: implement get namespaces route

---
 api/v1/models.go | 29 +++++++++++++++++------------
 api/v1/routes.go | 23 +++++++++++++++++------
 2 files changed, 34 insertions(+), 18 deletions(-)

diff --git a/api/v1/models.go b/api/v1/models.go
index a55870c1..5705197a 100644
--- a/api/v1/models.go
+++ b/api/v1/models.go
@@ -18,14 +18,9 @@ type Error struct {
 	Message string `json:"Layer`
 }
 
-type LayerEnvelope struct {
-	Layer *Layer `json:"Layer,omitempty"`
-	Error *Error `json:"Error,omitempty"`
-}
-
 type Layer struct {
 	Name             string    `json:"Name,omitempty"`
-	NamespaceName    string    `json:"NamespaceName,omitempty"`
+	Namespace        string    `json:"Namespace,omitempty"`
 	Path             string    `json:"Path,omitempty"`
 	ParentName       string    `json:"ParentName,omitempty"`
 	Format           string    `json:"Format,omitempty"`
@@ -34,12 +29,12 @@ type Layer struct {
 }
 
 type Vulnerability struct {
-	Name          string    `json:"Name,omitempty"`
-	NamespaceName string    `json:"NamespaceName,omitempty"`
-	Description   string    `json:"Description,omitempty"`
-	Severity      string    `json:"Severity,omitempty"`
-	FixedBy       string    `json:"FixedBy,omitempty"`
-	FixedIn       []Feature `json:"FixedIn,omitempty"`
+	Name        string    `json:"Name,omitempty"`
+	Namespace   string    `json:"Namespace,omitempty"`
+	Description string    `json:"Description,omitempty"`
+	Severity    string    `json:"Severity,omitempty"`
+	FixedBy     string    `json:"FixedBy,omitempty"`
+	FixedIn     []Feature `json:"FixedIn,omitempty"`
 }
 
 type Feature struct {
@@ -66,3 +61,13 @@ type VulnerabilityWithLayers struct {
 	Vulnerability                  *Vulnerability `json:"Vulnerability,omitempty"`
 	LayersIntroducingVulnerability []string       `json:"LayersIntroducingVulnerability,omitempty"`
 }
+
+type LayerEnvelope struct {
+	Layer *Layer `json:"Layer,omitempty"`
+	Error *Error `json:"Error,omitempty"`
+}
+
+type NamespaceEnvelope struct {
+	Namespaces *[]string `json:"Namespaces,omitempty"`
+	Error      *Error    `json:"Error,omitempty"`
+}
diff --git a/api/v1/routes.go b/api/v1/routes.go
index b117297d..739763dc 100644
--- a/api/v1/routes.go
+++ b/api/v1/routes.go
@@ -92,7 +92,7 @@ func getLayer(w http.ResponseWriter, r *http.Request, p httprouter.Params, ctx *
 	}
 
 	if dbLayer.Namespace != nil {
-		layer.NamespaceName = dbLayer.Namespace.Name
+		layer.Namespace = dbLayer.Namespace.Name
 	}
 
 	if withFeatures || withVulnerabilities && dbLayer.Features != nil {
@@ -105,10 +105,10 @@ func getLayer(w http.ResponseWriter, r *http.Request, p httprouter.Params, ctx *
 
 			for _, dbVuln := range dbFeatureVersion.AffectedBy {
 				vuln := Vulnerability{
-					Name:          dbVuln.Name,
-					NamespaceName: dbVuln.Namespace.Name,
-					Description:   dbVuln.Description,
-					Severity:      string(dbVuln.Severity),
+					Name:        dbVuln.Name,
+					Namespace:   dbVuln.Namespace.Name,
+					Description: dbVuln.Description,
+					Severity:    string(dbVuln.Severity),
 				}
 
 				if dbVuln.FixedBy != types.MaxVersion {
@@ -138,7 +138,18 @@ func deleteLayer(w http.ResponseWriter, r *http.Request, p httprouter.Params, ct
 }
 
 func getNamespaces(w http.ResponseWriter, r *http.Request, p httprouter.Params, ctx *context.RouteContext) int {
-	return 0
+	dbNamespaces, err := ctx.Store.ListNamespaces()
+	if err != nil {
+		writeResponse(w, NamespaceEnvelope{Error: &Error{err.Error()}})
+		return writeHeader(w, http.StatusInternalServerError)
+	}
+	var namespaces []string
+	for _, dbNamespace := range dbNamespaces {
+		namespaces = append(namespaces, dbNamespace.Name)
+	}
+
+	writeResponse(w, NamespaceEnvelope{Namespaces: &namespaces})
+	return writeHeader(w, http.StatusOK)
 }
 
 func postVulnerability(w http.ResponseWriter, r *http.Request, p httprouter.Params, ctx *context.RouteContext) int {