commands: use the new endpoint to detect boards

Author: alessio-perugini

commands/service_board_identify.go

@@ -130,7 +130,7 @@ func identifyViaCloudAPI(ctx context.Context, props *properties.Map, settings *c
 }
 
 var (
-	vidPidURL   = "https://builder.arduino.cc/v3/boards/byVidPid"
+	vidPidURL   = "https://api2.arduino.cc/boards/v1/boards"
 	validVidPid = regexp.MustCompile(`0[xX][a-fA-F\d]{4}`)
 )
 
@@ -169,7 +169,7 @@ func apiByVidPid(ctx context.Context, vid, pid string, settings *configuration.S
 		return nil, errors.New(i18n.Tr("Invalid pid value: '%s'", pid))
 	}
 
-	url := fmt.Sprintf("%s/%s/%s", vidPidURL, vid, pid)
+	url := fmt.Sprintf("%s?vid-pid=%s-%s", vidPidURL, vid, pid)
 	req, _ := http.NewRequest("GET", url, nil)
 	req.Header.Set("Content-Type", "application/json")
 
@@ -198,20 +198,23 @@ func apiByVidPid(ctx context.Context, vid, pid string, settings *configuration.S
 		return nil, err
 	}
 
-	var dat map[string]interface{}
+	type boardsResponse struct {
+		Items []struct {
+			Name string `json:"name"`
+			FQBN string `json:"fqbn"`
+		} `json:"items"`
+	}
+	var dat boardsResponse
 	if err := json.Unmarshal(resp, &dat); err != nil {
 		return nil, fmt.Errorf("%s: %w", i18n.Tr("error processing response from server"), err)
 	}
-	name, nameFound := dat["name"].(string)
-	fqbn, fbqnFound := dat["fqbn"].(string)
-	if !nameFound || !fbqnFound {
-		return nil, errors.New(i18n.Tr("wrong format in server response"))
-	}
 
-	return []*rpc.BoardListItem{
-		{
-			Name: name,
-			Fqbn: fqbn,
-		},
-	}, nil
+	response := make([]*rpc.BoardListItem, len(dat.Items))
+	for i, v := range dat.Items {
+		if v.Name == "" || v.FQBN == "" {
+			return nil, errors.New(i18n.Tr("wrong format in server response"))
+		}
+		response[i] = &rpc.BoardListItem{Name: v.Name, Fqbn: v.FQBN}
+	}
+	return response, nil
 }

commands/service_board_identify_test.go

@@ -35,25 +35,27 @@ func TestGetByVidPid(t *testing.T) {
 	ts := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
 		fmt.Fprintln(w, `
 {
-	"architecture": "samd",
-	"fqbn": "arduino:samd:mkr1000",
-	"href": "/v3/boards/arduino:samd:mkr1000",
-	"id": "mkr1000",
-	"name": "Arduino/Genuino MKR1000",
-	"package": "arduino",
-	"plan": "create-free"
+  "items": [
+    {
+      "fqbn": "arduino:avr:uno",
+      "vendor": "arduino",
+      "architecture": "avr",
+      "board_id": "uno",
+      "name": "Arduino Uno"
+    }
+  ]
 }
 		`)
 	}))
 	defer ts.Close()
 
 	vidPidURL = ts.URL
 	settings := configuration.NewSettings()
-	res, err := apiByVidPid(context.Background(), "0xf420", "0XF069", settings)
+	res, err := apiByVidPid(context.Background(), "0x2341", "0x0043", settings)
 	require.Nil(t, err)
 	require.Len(t, res, 1)
-	require.Equal(t, "Arduino/Genuino MKR1000", res[0].GetName())
-	require.Equal(t, "arduino:samd:mkr1000", res[0].GetFqbn())
+	require.Equal(t, "Arduino Uno", res[0].GetName())
+	require.Equal(t, "arduino:avr:uno", res[0].GetFqbn())
 
 	// wrong vid (too long), wrong pid (not an hex value)
 
@@ -95,7 +97,7 @@ func TestGetByVidPidMalformedResponse(t *testing.T) {
 	settings := configuration.NewSettings()
 
 	ts := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-		fmt.Fprintln(w, "{}")
+		fmt.Fprintln(w, `{"items":[{}]}`)
 	}))
 	defer ts.Close()