chore: use patch not put

env/prod

@@ -1,4 +1,4 @@
-API_URL=https://api.azion.net
+API_URL=https://api.azionapi.net
 API_V4_URL=https://api.azion.com/v4
 AUTH_URL=https://sso.azion.com/api
 STORAGE_URL=https://api.azion.com/v4

pkg/api/device_groups/device_groups.go

@@ -30,12 +30,12 @@ func (c *Client) Create(ctx context.Context, req sdk.DeviceGroupRequest, applica
 	return resp.GetData(), nil
 }
 
-func (c *Client) Update(ctx context.Context, req sdk.DeviceGroupRequest, applicationID, deviceGroupID int64) (sdk.DeviceGroup, error) {
+func (c *Client) Update(ctx context.Context, req sdk.PatchedDeviceGroupRequest, applicationID, deviceGroupID int64) (sdk.DeviceGroup, error) {
 	logger.Debug("Update Device Group")
 
 	request := c.apiClient.ApplicationsDeviceGroupsAPI.
-		UpdateDeviceGroup(ctx, applicationID, deviceGroupID).
-		DeviceGroupRequest(req)
+		PartialUpdateDeviceGroup(ctx, applicationID, deviceGroupID).
+		PatchedDeviceGroupRequest(req)
 	resp, httpResp, err := request.Execute()
 	if err != nil {
 		logger.Debug("Error while updating a Device Group", zap.Any("ID", deviceGroupID), zap.Error(err))

pkg/api/dns_record/dns_record.go

@@ -30,12 +30,12 @@ func (c *Client) Create(ctx context.Context, req sdk.RecordRequest, zoneID int64
 	return resp.GetData(), nil
 }
 
-func (c *Client) Update(ctx context.Context, req sdk.RecordRequest, zoneID, recordID int64) (sdk.Record, error) {
+func (c *Client) Update(ctx context.Context, req sdk.PatchedRecordRequest, zoneID, recordID int64) (sdk.Record, error) {
 	logger.Debug("Update DNS Record")
 
 	request := c.apiClient.DNSRecordsAPI.
-		UpdateDnsRecord(ctx, recordID, zoneID).
-		RecordRequest(req)
+		PartialUpdateDnsRecord(ctx, recordID, zoneID).
+		PatchedRecordRequest(req)
 	resp, httpResp, err := request.Execute()
 	if err != nil {
 		logger.Debug("Error while updating a DNS Record", zap.Any("ID", recordID), zap.Error(err))

pkg/api/dns_zone/dns_zone.go

@@ -30,12 +30,12 @@ func (c *Client) Create(ctx context.Context, req sdk.ZoneRequest) (sdk.Zone, err
 	return resp.GetData(), nil
 }
 
-func (c *Client) Update(ctx context.Context, req sdk.UpdateZoneRequest, zoneID int64) (sdk.Zone, error) {
+func (c *Client) Update(ctx context.Context, req sdk.PatchedUpdateZoneRequest, zoneID int64) (sdk.Zone, error) {
 	logger.Debug("Update DNS Zone")
 
 	request := c.apiClient.DNSZonesAPI.
-		UpdateDnsZone(ctx, zoneID).
-		UpdateZoneRequest(req)
+		PartialUpdateDnsZone(ctx, zoneID).
+		PatchedUpdateZoneRequest(req)
 	resp, httpResp, err := request.Execute()
 	if err != nil {
 		logger.Debug("Error while updating a DNS Zone", zap.Any("ID", zoneID), zap.Error(err))

pkg/cmd/update/device_groups/device_groups.go

@@ -43,7 +43,7 @@ func NewCmd(f *cmdutil.Factory) *cobra.Command {
 		RunE: func(cmd *cobra.Command, args []string) error {
 			client := api.NewClient(f.HttpClient, f.Config.GetString("api_v4_url"), f.Config.GetString("token"))
 
-			request := sdk.DeviceGroupRequest{}
+			request := sdk.PatchedDeviceGroupRequest{}
 
 			if !cmd.Flags().Changed("application-id") {
 				answers, err := utils.AskInput(msg.DeviceGroupsUpdateAskInputApplicationID)
@@ -84,9 +84,6 @@ func NewCmd(f *cmdutil.Factory) *cobra.Command {
 					return utils.ErrorUnmarshalReader
 				}
 			} else {
-				// The device group update is a full replacement (PUT), and both
-				// name and user_agent are required. Fetch the current values for
-				// any attribute the user didn't provide so they are preserved.
 				if !cmd.Flags().Changed("name") || !cmd.Flags().Changed("user-agent") {
 					current, err := client.Get(context.Background(), fields.ApplicationID, fields.GroupID)
 					if err != nil {

pkg/cmd/update/dns_record/dns_record.go

@@ -49,7 +49,7 @@ func NewCmd(f *cmdutil.Factory) *cobra.Command {
 		RunE: func(cmd *cobra.Command, args []string) error {
 			client := api.NewClient(f.HttpClient, f.Config.GetString("api_v4_url"), f.Config.GetString("token"))
 
-			request := sdk.RecordRequest{}
+			request := sdk.PatchedRecordRequest{}
 
 			if !cmd.Flags().Changed("zone-id") {
 				answers, err := utils.AskInput(msg.DNSRecordUpdateAskInputZoneID)
@@ -90,9 +90,8 @@ func NewCmd(f *cmdutil.Factory) *cobra.Command {
 					return utils.ErrorUnmarshalReader
 				}
 			} else {
-				// The DNS record update is a full replacement (PUT); name, type
-				// and rdata are required. Fetch the current record and preserve
-				// any attribute the user didn't provide.
+				// The update is a partial update (PATCH). Fetch the current
+				// record and preserve any attribute the user didn't provide.
 				current, err := client.Get(context.Background(), fields.ZoneID, fields.RecordID)
 				if err != nil {
 					return fmt.Errorf(msg.ErrorGetDNSRecord.Error(), err)

pkg/cmd/update/dns_record/dns_record_test.go

@@ -25,7 +25,7 @@ func TestUpdate(t *testing.T) {
 			httpmock.JSONFromFile("./fixtures/response.json"),
 		)
 		mock.Register(
-			httpmock.REST("PUT", "workspace/dns/zones/1337/records/1111"),
+			httpmock.REST("PATCH", "workspace/dns/zones/1337/records/1111"),
 			httpmock.JSONFromFile("./fixtures/response.json"),
 		)
 
@@ -42,7 +42,7 @@ func TestUpdate(t *testing.T) {
 	t.Run("update with file", func(t *testing.T) {
 		mock := &httpmock.Registry{}
 		mock.Register(
-			httpmock.REST("PUT", "workspace/dns/zones/1337/records/1111"),
+			httpmock.REST("PATCH", "workspace/dns/zones/1337/records/1111"),
 			httpmock.JSONFromFile("./fixtures/response.json"),
 		)
 

pkg/cmd/update/dns_zone/dns_zone.go

@@ -42,7 +42,7 @@ func NewCmd(f *cmdutil.Factory) *cobra.Command {
 		RunE: func(cmd *cobra.Command, args []string) error {
 			client := api.NewClient(f.HttpClient, f.Config.GetString("api_v4_url"), f.Config.GetString("token"))
 
-			request := sdk.UpdateZoneRequest{}
+			request := sdk.PatchedUpdateZoneRequest{}
 
 			if !cmd.Flags().Changed("zone-id") {
 				answers, err := utils.AskInput(msg.DNSZoneUpdateAskInputZoneID)
@@ -67,9 +67,9 @@ func NewCmd(f *cmdutil.Factory) *cobra.Command {
 					return utils.ErrorUnmarshalReader
 				}
 			} else {
-				// The DNS zone update is a full replacement (PUT), and both
-				// name and active are required. Fetch the current values for
-				// any attribute the user didn't provide so they are preserved.
+				// The update is a partial update (PATCH). Fetch the current
+				// values for any attribute the user didn't provide so they are
+				// preserved.
 				if !cmd.Flags().Changed("name") || !cmd.Flags().Changed("active") {
 					current, err := client.Get(context.Background(), fields.ZoneID)
 					if err != nil {

pkg/cmd/update/dns_zone/dns_zone_test.go

@@ -20,7 +20,7 @@ func TestUpdate(t *testing.T) {
 	t.Run("update DNS zone with name and active", func(t *testing.T) {
 		mock := &httpmock.Registry{}
 		mock.Register(
-			httpmock.REST("PUT", "workspace/dns/zones/1337"),
+			httpmock.REST("PATCH", "workspace/dns/zones/1337"),
 			httpmock.JSONFromFile("./fixtures/response.json"),
 		)
 
@@ -42,7 +42,7 @@ func TestUpdate(t *testing.T) {
 			httpmock.JSONFromFile("./fixtures/response.json"),
 		)
 		mock.Register(
-			httpmock.REST("PUT", "workspace/dns/zones/1337"),
+			httpmock.REST("PATCH", "workspace/dns/zones/1337"),
 			httpmock.JSONFromFile("./fixtures/response.json"),
 		)
 
@@ -59,7 +59,7 @@ func TestUpdate(t *testing.T) {
 	t.Run("update with file", func(t *testing.T) {
 		mock := &httpmock.Registry{}
 		mock.Register(
-			httpmock.REST("PUT", "workspace/dns/zones/1337"),
+			httpmock.REST("PATCH", "workspace/dns/zones/1337"),
 			httpmock.JSONFromFile("./fixtures/response.json"),
 		)
 
@@ -76,7 +76,7 @@ func TestUpdate(t *testing.T) {
 	t.Run("not found", func(t *testing.T) {
 		mock := &httpmock.Registry{}
 		mock.Register(
-			httpmock.REST("PUT", "workspace/dns/zones/1234"),
+			httpmock.REST("PATCH", "workspace/dns/zones/1234"),
 			httpmock.StatusStringResponse(http.StatusNotFound, "Not Found"),
 		)
 

utils/helpers.go

@@ -799,6 +799,10 @@ func LogAndRewindBody(httpResp *http.Response) error {
 }
 
 func LogAndRewindBodyV4(httpResp *http.Response) (string, error) {
+	if httpResp == nil {
+		return "", nil
+	}
+
 	logger.Debug("", zap.Any("Status Code", httpResp.StatusCode))
 	logger.Debug("", zap.Any("Headers", httpResp.Header))
 	var errResp ErrorResponse