OMG-25737 - Add SubAccount type and CreateSubAccount (#8)

Author: slavick

client/carrier_account.go

@@ -20,6 +20,16 @@ func (c *Client) CreateCarrierAccount(input *models.CarrierAccountInput, shippoS
 	return output, err
 }
 
+func (c *Client) RegisterCarrierAccount(input *models.CarrierAccountInput, shippoSubAccountID string) (*models.CarrierAccount, error) {
+	if input == nil {
+		return nil, errors.New("nil input")
+	}
+
+	output := &models.CarrierAccount{}
+	err := c.doWithoutVersion(http.MethodPost, "/carrier_accounts/register/new", input, output, c.subAccountHeader(shippoSubAccountID))
+	return output, err
+}
+
 // RetrieveCarrierAccount retrieves an existing carrier account by object id.
 func (c *Client) RetrieveCarrierAccount(objectID string, shippoSubAccountID string) (*models.CarrierAccount, error) {
 	if objectID == "" {

client/client.go

@@ -13,7 +13,8 @@ import (
 )
 
 const (
-	shippoAPIBaseURL = "https://api.goshippo.com/v1"
+	shippoAPIBaseURL          = "https://api.goshippo.com/v1"
+	shippoAPIBaseURLNoVersion = "https://api.goshippo.com"
 )
 
 type Client struct {
@@ -41,7 +42,17 @@ func (c *Client) SetTraceLogger(logger *log.Logger) *log.Logger {
 }
 
 func (c *Client) do(method, path string, input, output interface{}, headers map[string]string) error {
-	url := shippoAPIBaseURL + path
+	return c._do(shippoAPIBaseURL, method, path, input, output, headers)
+}
+
+// doWithoutVersion was added to support the /shippo-accounts endpoint that throws a 404 if `/v1/` is present
+// https://docs.goshippo.com/docs/platformaccounts/platform_using_accounts/#create-a-managed-shippo-account-for-your-customer
+func (c *Client) doWithoutVersion(method, path string, input, output interface{}, headers map[string]string) error {
+	return c._do(shippoAPIBaseURLNoVersion, method, path, input, output, headers)
+}
+
+func (c *Client) _do(baseUrl, method, path string, input, output interface{}, headers map[string]string) error {
+	url := baseUrl + path
 
 	req, err := c.createRequest(method, url, input, headers)
 	if err != nil {
@@ -105,8 +116,10 @@ func (c *Client) createRequest(method, url string, bodyObject interface{}, heade
 			}
 
 			for hk, hva := range req.Header {
-				for _, hv := range hva {
-					c.logPrintf("Client.createRequest() Header %s=%s", hk, hv)
+				if hk != "Authorization" {
+					for _, hv := range hva {
+						c.logPrintf("Client.createRequest() Header %s=%s", hk, hv)
+					}
 				}
 			}
 

client/sub_account.go

@@ -0,0 +1,17 @@
+package client
+
+import (
+	"errors"
+	"github.com/OrderMyGear/go-shippo/models"
+	"net/http"
+)
+
+func (c *Client) CreateSubAccount(input *models.SubAccountInput) (*models.SubAccount, error) {
+	if input == nil {
+		return nil, errors.New("nil input")
+	}
+
+	output := &models.SubAccount{}
+	err := c.doWithoutVersion(http.MethodPost, "/shippo-accounts", input, output, nil)
+	return output, err
+}

models/sub_account.go

@@ -0,0 +1,15 @@
+package models
+
+type SubAccountInput struct {
+	Email       string `json:"email"`
+	FirstName   string `json:"first_name"`
+	LastName    string `json:"last_name"`
+	CompanyName string `json:"company_name"`
+}
+
+type SubAccount struct {
+	SubAccountInput
+	CommonOutputFields
+	Test       bool        `json:"test"`
+	ObjectInfo *ObjectInfo `json:"object_info"`
+}