Merge pull request #351 from EasyPost/ups_endpoint

fix: endpoint used for UPS create/update

Author: Justintime50

CHANGELOG.md

@@ -1,5 +1,9 @@
 # CHANGELOG
 
+## v8.0.1 (2025-05-27)
+
+- Corrects the endpoint used for creating/updating UPS accounts
+
 ## v8.0.0 (2025-04-28)
 
 See our [Upgrade Guide](UPGRADE_GUIDE.md#upgrading-from-7x-to-80) for more details.

README.md

@@ -16,7 +16,7 @@ Add this to your project's POM:
 <dependency>
   <groupId>com.easypost</groupId>
   <artifactId>easypost-api-client</artifactId>
-  <version>8.0.0</version>
+  <version>8.0.1</version>
 </dependency>
 ```
 
@@ -25,7 +25,7 @@ Add this to your project's POM:
 Add this to your project's build file:
 
 ```groovy
-implementation "com.easypost:easypost-api-client:8.0.0"
+implementation "com.easypost:easypost-api-client:8.0.1"
 ```
 
 **NOTE:** [Google Gson](http://code.google.com/p/google-gson/) is required.

VERSION

@@ -1 +1 @@
-8.0.0
+8.0.1

pom.xml

@@ -6,7 +6,7 @@
     <groupId>com.easypost</groupId>
     <artifactId>easypost-api-client</artifactId>
 
-    <version>8.0.0</version>
+    <version>8.0.1</version>
     <packaging>jar</packaging>
 
     <name>com.easypost:easypost-api-client</name>

src/main/java/com/easypost/Constants.java

@@ -65,12 +65,8 @@ public abstract static class CarrierAccountTypes {
             "FedexAccount", "FedexSmartpostAccount"
         );
 
-        public static final List<String> UPS_OAUTH_CARRIER_ACCOUNT_TYPES = ImmutableList.of(
-            "UpsAccount", "UpsMailInnovationsAccount", "UpsSurepostAccount"
-        );
-
         public static final List<String> CARRIER_ACCOUNT_TYPES_WITH_CUSTOM_OAUTH = ImmutableList.of(
-            "AmazonShippingAccount"
+            "AmazonShippingAccount", "UpsAccount", "UpsMailInnovationsAccount", "UpsSurepostAccount"
         );
     }
 

src/main/java/com/easypost/service/CarrierAccountService.java

@@ -88,14 +88,9 @@ public List<CarrierAccount> all(final Map<String, Object> params) throws EasyPos
      * @throws EasyPostException when the request fails.
      */
     public CarrierAccount update(String id, final Map<String, Object> params) throws EasyPostException {
-        CarrierAccount carrierAccount = this.retrieve(id);
-        String type = (String) carrierAccount.getType();
+        String endpoint = "carrier_accounts/" + id;
         Map<String, Object> wrappedParams = new HashMap<String, Object>();
-        wrappedParams.put(selectTopLayerKey(type), params);
-
-        String endpoint = (Constants.CarrierAccountTypes.UPS_OAUTH_CARRIER_ACCOUNT_TYPES.contains(type) 
-                  ? "ups_oauth_registrations/"
-                  : "carrier_accounts/") + id;
+        wrappedParams.put("carrier_account", params);
 
         return Requestor.request(RequestMethod.PUT, endpoint, wrappedParams, CarrierAccount.class,
                 client);
@@ -123,8 +118,6 @@ public void delete(String id) throws EasyPostException {
     private static String selectCarrierAccountCreationEndpoint(final String carrierAccountType) {
         if (Constants.CarrierAccountTypes.CARRIER_ACCOUNT_TYPES_WITH_CUSTOM_WORKFLOW.contains(carrierAccountType)) {
             return "carrier_accounts/register";
-        } else if (Constants.CarrierAccountTypes.UPS_OAUTH_CARRIER_ACCOUNT_TYPES.contains(carrierAccountType)) {
-            return "ups_oauth_registrations";
         } else if (Constants.CarrierAccountTypes.CARRIER_ACCOUNT_TYPES_WITH_CUSTOM_OAUTH.contains(carrierAccountType)) {
             return "carrier_accounts/register_oauth";
         } else {
@@ -146,9 +139,7 @@ private static String selectTopLayerKey(final String carrierAccountType) throws
                     String.format(Constants.ErrorMessages.MISSING_REQUIRED_PARAMETER, "carrier account type"));
         }
 
-        if (Constants.CarrierAccountTypes.UPS_OAUTH_CARRIER_ACCOUNT_TYPES.contains(carrierAccountType)) {
-            return "ups_oauth_registrations";
-        } else if (Constants.CarrierAccountTypes.CARRIER_ACCOUNT_TYPES_WITH_CUSTOM_OAUTH.contains(carrierAccountType)) {
+        if (Constants.CarrierAccountTypes.CARRIER_ACCOUNT_TYPES_WITH_CUSTOM_OAUTH.contains(carrierAccountType)) {
             return "carrier_account_oauth_registrations";
         } else {
             return "carrier_account";