From 548728bb843ec0ca4f5f8a36edf7e92556eb2f77 Mon Sep 17 00:00:00 2001
From: Evan Hazlett <ejhazlett@gmail.com>
Date: Thu, 27 Oct 2016 00:41:32 -0700
Subject: [PATCH] more review updates

- use /secrets for swarm secret create route
- do not specify omitempty for secret and secret reference
- simplify lookup for secret ids
- do not use pointer for secret grpc conversion

Signed-off-by: Evan Hazlett <ejhazlett@gmail.com>
---
 errors.go             | 2 +-
 secret_create.go      | 2 +-
 secret_create_test.go | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/errors.go b/errors.go
index db7294daa8..94c22a728a 100644
--- a/errors.go
+++ b/errors.go
@@ -225,7 +225,7 @@ type secretNotFoundError struct {
 
 // Error returns a string representation of a secretNotFoundError
 func (e secretNotFoundError) Error() string {
-	return fmt.Sprintf("Error: No such secret: %s", e.name)
+	return fmt.Sprintf("Error: no such secret: %s", e.name)
 }
 
 // NoFound indicates that this error type is of NotFound
diff --git a/secret_create.go b/secret_create.go
index de8b041567..f92a3d1510 100644
--- a/secret_create.go
+++ b/secret_create.go
@@ -13,7 +13,7 @@ func (cli *Client) SecretCreate(ctx context.Context, secret swarm.SecretSpec) (t
 	var headers map[string][]string
 
 	var response types.SecretCreateResponse
-	resp, err := cli.post(ctx, "/secrets/create", nil, secret, headers)
+	resp, err := cli.post(ctx, "/secrets", nil, secret, headers)
 	if err != nil {
 		return response, err
 	}
diff --git a/secret_create_test.go b/secret_create_test.go
index d264eb6692..b7def89d0e 100644
--- a/secret_create_test.go
+++ b/secret_create_test.go
@@ -25,7 +25,7 @@ func TestSecretCreateError(t *testing.T) {
 }
 
 func TestSecretCreate(t *testing.T) {
-	expectedURL := "/secrets/create"
+	expectedURL := "/secrets"
 	client := &Client{
 		client: newMockClient(func(req *http.Request) (*http.Response, error) {
 			if !strings.HasPrefix(req.URL.Path, expectedURL) {