Update status code for CRUD APIs

This commit updates the status code from 200 to 202 for create, update
or delete APIs.

Fixes: openfaas#220

Signed-off-by: Vivek Singh <[email protected]>

Author: viveksyngh

handlers/delete.go

@@ -59,6 +59,8 @@ func MakeDeleteHandler(functionNamespace string, clientset *kubernetes.Clientset
 			w.Write([]byte("Not a function: " + request.FunctionName))
 			return
 		}
+
+		w.WriteHeader(http.StatusAccepted)
 	}
 }
 

handlers/replicas.go

@@ -63,6 +63,7 @@ func MakeReplicaUpdater(functionNamespace string, clientset *kubernetes.Clientse
 			return
 		}
 
+		w.WriteHeader(http.StatusAccepted)
 	}
 }
 
@@ -81,13 +82,13 @@ func MakeReplicaReader(functionNamespace string, clientset *kubernetes.Clientset
 		}
 
 		if function == nil {
-			w.WriteHeader(404)
+			w.WriteHeader(http.StatusNotFound)
 			return
 		}
 
 		functionBytes, _ := json.Marshal(function)
 		w.Header().Set("Content-Type", "application/json")
-		w.WriteHeader(200)
+		w.WriteHeader(http.StatusOK)
 		w.Write(functionBytes)
 	}
 }

handlers/update.go

@@ -38,6 +38,8 @@ func MakeUpdateHandler(functionNamespace string, clientset *kubernetes.Clientset
 			w.WriteHeader(status)
 			w.Write([]byte(err.Error()))
 		}
+
+		w.WriteHeader(http.StatusAccepted)
 	}
 }
 
@@ -118,7 +120,7 @@ func updateDeploymentSpec(
 		return updateErr, http.StatusInternalServerError
 	}
 
-	return nil, http.StatusOK
+	return nil, http.StatusAccepted
 }
 
 func updateService(
@@ -146,5 +148,5 @@ func updateService(
 		return updateErr, http.StatusInternalServerError
 	}
 
-	return nil, http.StatusOK
+	return nil, http.StatusAccepted
 }