OMG-25752 - Truncate fields before sending to Shippo (#11)

OMG-25752 - Truncate fields before sending to Shippo
OrderMyGear · Oct 24, 2024 · ade2e9e · ade2e9e
2 parents 3ede812 + a240d6c
commit ade2e9e
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 0 deletions.
diff --git a/client/address.go b/client/address.go
@@ -14,6 +14,11 @@ func (c *Client) CreateAddress(input *models.AddressInput, shippoSubAccountID st
 		return nil, errors.New("nil input")
 	}
 
+	// Truncate phone to 50 characters to avoid Stripe API errors
+	if len(input.Phone) > 50 {
+		input.Phone = input.Phone[:50]
+	}
+
 	output := &models.Address{}
 	err := c.do(http.MethodPost, "/addresses/", input, output, c.subAccountHeader(shippoSubAccountID))
 	return output, err

diff --git a/client/sub_account.go b/client/sub_account.go
@@ -11,6 +11,17 @@ func (c *Client) CreateSubAccount(input *models.SubAccountInput) (*models.SubAcc
 		return nil, errors.New("nil input")
 	}
 
+	// Truncate fields to avoid Stripe API errors
+	if len(input.FirstName) > 30 {
+		input.FirstName = input.FirstName[:30]
+	}
+	if len(input.LastName) > 30 {
+		input.LastName = input.LastName[:30]
+	}
+	if len(input.CompanyName) > 50 {
+		input.CompanyName = input.CompanyName[:50]
+	}
+
 	output := &models.SubAccount{}
 	err := c.doWithoutVersion(http.MethodPost, "/shippo-accounts", input, output, nil)
 	return output, err