Skip to content

Commit

Permalink
MF-1833 - Fix spelling of bootstrap on SDK (#1834)
Browse files Browse the repository at this point in the history
* fix order of parameters and add url checks

Signed-off-by: SammyOina <sammyoina@gmail.com>

* fix spelling in sdk

Signed-off-by: SammyOina <sammyoina@gmail.com>

* wrap jaeger url parsing error

Signed-off-by: SammyOina <sammyoina@gmail.com>

* remove jaeger

Signed-off-by: SammyOina <sammyoina@gmail.com>

* remove empty lines

Signed-off-by: SammyOina <sammyoina@gmail.com>

---------

Signed-off-by: SammyOina <sammyoina@gmail.com>
  • Loading branch information
SammyOina committed Jul 13, 2023
1 parent f7ba155 commit 52131eb
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 10 deletions.
13 changes: 5 additions & 8 deletions pkg/sdk/go/bootstrap.go
Original file line number Diff line number Diff line change
Expand Up @@ -66,20 +66,20 @@ func (sdk mfSDK) AddBootstrap(cfg BootstrapConfig, token string) (string, errors
return id, nil
}

func (sdk mfSDK) Bootstraps(pm PageMetadata, token string) (BoostrapsPage, errors.SDKError) {
func (sdk mfSDK) Bootstraps(pm PageMetadata, token string) (BootstrapPage, errors.SDKError) {
url, err := sdk.withQueryParams(sdk.bootstrapURL, configsEndpoint, pm)
if err != nil {
return BoostrapsPage{}, errors.NewSDKError(err)
return BootstrapPage{}, errors.NewSDKError(err)
}

_, body, sdkerr := sdk.processRequest(http.MethodGet, url, token, string(CTJSON), nil, http.StatusOK)
if sdkerr != nil {
return BoostrapsPage{}, sdkerr
return BootstrapPage{}, sdkerr
}

var bb BoostrapsPage
var bb BootstrapPage
if err = json.Unmarshal(body, &bb); err != nil {
return BoostrapsPage{}, errors.NewSDKError(err)
return BootstrapPage{}, errors.NewSDKError(err)
}

return bb, nil
Expand Down Expand Up @@ -143,7 +143,6 @@ func (sdk mfSDK) UpdateBootstrapCerts(id, clientCert, clientKey, ca, token strin
}

_, _, sdkerr := sdk.processRequest(http.MethodPatch, url, token, string(CTJSON), data, http.StatusOK)

return sdkerr
}

Expand All @@ -158,14 +157,12 @@ func (sdk mfSDK) UpdateBootstrapConnection(id string, channels []string, token s
}

_, _, sdkerr := sdk.processRequest(http.MethodPut, url, token, string(CTJSON), data, http.StatusOK)

return sdkerr
}

func (sdk mfSDK) RemoveBootstrap(id, token string) errors.SDKError {
url := fmt.Sprintf("%s/%s/%s", sdk.bootstrapURL, configsEndpoint, id)
_, _, err := sdk.processRequest(http.MethodDelete, url, token, string(CTJSON), nil, http.StatusNoContent)

return err
}

Expand Down
2 changes: 1 addition & 1 deletion pkg/sdk/go/responses.go
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,7 @@ type revokeCertsRes struct {
}

// BoostrapsPage contains list of boostrap configs in a page with proper metadata.
type BoostrapsPage struct {
type BootstrapPage struct {
Configs []BootstrapConfig `json:"configs"`
pageRes
}
Expand Down
2 changes: 1 addition & 1 deletion pkg/sdk/go/sdk.go
Original file line number Diff line number Diff line change
Expand Up @@ -830,7 +830,7 @@ type SDK interface {
// }
// bootstraps, _ := sdk.Bootstraps(pm, "token")
// fmt.Println(bootstraps)
Bootstraps(pm PageMetadata, token string) (BoostrapsPage, errors.SDKError)
Bootstraps(pm PageMetadata, token string) (BootstrapPage, errors.SDKError)

// Whitelist updates Thing state Config with given ID belonging to the user identified by the given token.
//
Expand Down

0 comments on commit 52131eb

Please sign in to comment.