Merge branch 'release-24.5' into chore-fw-port-23-5

Author: lucamarchiori

.github/workflows/go-mod-tidy.yaml

@@ -24,7 +24,7 @@ jobs:
     - uses: actions/checkout@v3
     - uses: actions/setup-go@v4
       with:
-        go-version: v1.20.x
+        go-version: v1.21.x
     - name: go-mod-tidy
       run: |
         go mod tidy

.github/workflows/golangci-lint.yaml

@@ -1,4 +1,4 @@
-# Copyright © 2023 SECO Mind Srl
+# Copyright © 2023-2024 SECO Mind Srl
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.
@@ -25,8 +25,8 @@ jobs:
     # Since v3, golangci-lint needs explicit go-setup
     - uses: actions/setup-go@v4
       with:
-        go-version: v1.20.x
+        go-version: v1.21.x
     # Run golint-ci
     - uses: golangci/golangci-lint-action@v3
       with:
-        version: v1.53
+        version: v1.59

.github/workflows/goreleaser.yml

@@ -14,15 +14,15 @@ jobs:
       uses: actions/checkout@v3
       with:
         fetch-depth: 0
-    - name: Set up Go 1.20
+    - name: Set up Go 1.21
       uses: actions/setup-go@v4
       with:
-        go-version: 1.20.x
+        go-version: 1.21.x
     - name: Run GoReleaser
-      uses: goreleaser/goreleaser-action@v4
+      uses: goreleaser/goreleaser-action@v6
       with:
         distribution: goreleaser
-        version: latest
+        version: v2
         args: release --clean
       env:
         GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

.github/workflows/main.yml

@@ -15,7 +15,7 @@ jobs:
     name: Build and Test
     strategy:
       matrix:
-        go: [1.19.x, 1.20.x]
+        go: [1.21.x, 1.22.x]
         os: [ubuntu-22.04, macos-latest, windows-latest]
     runs-on: ${{ matrix.os }}
     steps:
@@ -40,10 +40,9 @@ jobs:
     - name: Test
       run: go test -v -race ./...
 
-    # Load artifacts obtained with the most recent version of Go
     - name: Upload build artifacts
       uses: actions/upload-artifact@v4
-      if: matrix.go == '1.20.x'
+      if: matrix.go == '1.22.x'
       with:
         path: astartectl*
         name: astartectl-snapshot-${{ matrix.os }}

.tool-versions

@@ -1 +1 @@
-golang 1.20.5
+golang 1.21.11

CHANGELOG.md

@@ -4,6 +4,39 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/)
 and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html).
 
+## [24.5.3] - Unreleased
+### Changed
+- `appengine device attributes list`: display attributes in a human-friendly
+  representation instead Go's internal one. Keep consistency with aliases.
+
+### Fixed
+- `appengine device alias list`: show aliases as a map instead of a list.
+  Fix [#275](https://github.com/astarte-platform/astartectl/issues/275).
+
+## [24.5.2] - 2024-09-20
+### Fixed
+- Allow a larger set of permissions for configuration files and folders.
+
+## [24.5.1] - 2024-09-19
+### Changed
+- `cluster instance` {`deploy` | `destroy` | `show`} are deprecated and
+  will be removed from v24.11.
+
+### Fixed
+- Create configuration files and folders if required, instead of just crashing.
+- Fixed empty array sent as empty string for array payload.
+- Allow the management of triggers containing Mustache templates.
+
+## [24.5.0] - 2024-09-03
+### Added
+- `cluster instance deploy`: add profiles for astarte v1.2.
+
+### Changed
+- Use Go 1.21 for releases.
+
+### Removed
+- `cluster instance deploy`: remove profiles for Astarte v0.10 and v0.11.
+
 ## [23.5.2] - 2024-09-02
 ### Fixed
 - `realm-management {interfaces, triggers} sync`: do not show a wrong curl

cmd/appengine/device.go

@@ -21,6 +21,7 @@ import (
 	"fmt"
 	"math"
 	"os"
+	"regexp"
 	"strconv"
 	"strings"
 	"text/tabwriter"
@@ -1649,34 +1650,34 @@ func parseSendDataPayload(payload string, mappingType interfaces.AstarteMappingT
 	case interfaces.BinaryBlobArray, interfaces.BooleanArray, interfaces.DateTimeArray, interfaces.DoubleArray,
 		interfaces.IntegerArray, interfaces.LongIntegerArray, interfaces.StringArray:
 
-		//wait it's all string?
-		payload = strings.Replace(payload, "[", "", -1)
-		payload = strings.Replace(payload, "]", "", -1)
-		payload_parsed := strings.Split(payload, ",")
-		//always has been
-
-		jsonOut := make([]interface{}, len(payload_parsed))
-		for i, v := range payload_parsed {
-			jsonOut[i] = v
-		}
-
-		retArray := []interface{}{}
-		// Do a smarter conversion here.
-		for _, v := range jsonOut {
-			switch val := v.(type) {
-			case string:
-				p, err := parseSendDataPayload(strings.TrimSpace(val), interfaces.AstarteMappingType(strings.TrimSuffix(string(mappingType), "array")))
+		//check if payload is an empty array, bypass conversions and just return an empty array
+		// if it is not, proceed as usual
+		if payload == "[]" {
+			ret = make([]interface{}, 0)
+		} else {
+			retArray := []interface{}{}
+			payload = strings.TrimSpace(payload)
+			if pass, err := regexp.MatchString(`^\[.*\]$`, payload); !pass {
+				return nil, err
+			}
+			payload = regexp.MustCompile(`^\[`).ReplaceAllString(payload, "")
+			payload = regexp.MustCompile(`\]$`).ReplaceAllString(payload, "")
+			payload_parsed := strings.Split(payload, ",")
+			jsonOut := make([]interface{}, len(payload_parsed))
+			for i, v := range payload_parsed {
+				jsonOut[i] = v
+			}
+			// Do a smarter conversion here.
+			for _, v := range jsonOut {
+				p, err := parseSendDataPayload(strings.TrimSpace(v.(string)), interfaces.AstarteMappingType(strings.TrimSuffix(string(mappingType), "array")))
 				if err != nil {
 					return nil, err
 				}
 				retArray = append(retArray, p)
-			default:
-				retArray = append(retArray, val)
 			}
+			ret = retArray
 		}
-		ret = retArray
 	}
-
 	return ret, nil
 }
 

cmd/appengine/device_aliases.go

@@ -92,8 +92,12 @@ func aliasesListF(command *cobra.Command, args []string) error {
 		os.Exit(1)
 	}
 	aliases, _ := deviceAliasesRes.Parse()
+	aliasesMap, _ := aliases.(map[string]string)
+
+	for k, v := range aliasesMap {
+		fmt.Printf("%v: %v\n", k, v)
+	}
 
-	fmt.Printf("%v\n", aliases)
 	return nil
 }
 

cmd/appengine/device_attributes.go

@@ -111,8 +111,12 @@ func attributesListF(command *cobra.Command, args []string) error {
 		os.Exit(1)
 	}
 	attributes, _ := attributesRes.Parse()
+	attributesMap, _ := attributes.(map[string]string)
+
+	for k, v := range attributesMap {
+		fmt.Printf("%v: %v\n", k, v)
+	}
 
-	fmt.Printf("%v\n", attributes)
 	return nil
 }
 

cmd/cluster/deployment/all_profiles.go

@@ -17,16 +17,14 @@ package deployment
 // GetAllBuiltinAstarteClusterProfiles returns all the bundled Cluster profiles.
 func GetAllBuiltinAstarteClusterProfiles() []AstarteClusterProfile {
 	return []AstarteClusterProfile{
-		// 0.10 profiles
-		astarteBasicProfile010,
-		// 0.11 profiles
-		astarteBasicProfile011,
-		astarteBurstProfile011,
 		// 1.0 profiles (good for 1.0 for now)
 		astarteBasicProfile10,
 		astarteBurstProfile10,
 		// 1.1 profiles
 		astarteBasicProfile11,
 		astarteBurstProfile11,
+		// 1.2 profiles
+		astarteBasicProfile12,
+		astarteBurstProfile12,
 	}
 }