Handle case of missing OAuth provider

Signed-off-by: Sundeep Gottipati <[email protected]>

Author: kgprs

cmd/docker-mcp/internal/desktop/raw_client.go

@@ -4,6 +4,7 @@ import (
 	"bytes"
 	"context"
 	"encoding/json"
+	"fmt"
 	"io"
 	"net"
 	"net/http"
@@ -51,6 +52,12 @@ func (c *RawClient) Get(ctx context.Context, endpoint string, v any) error {
 	}
 	defer response.Body.Close()
 
+	// Check for HTTP errors
+	if response.StatusCode >= 400 {
+		buf, _ := io.ReadAll(response.Body)
+		return fmt.Errorf("HTTP %d: %s", response.StatusCode, string(buf))
+	}
+
 	buf, err := io.ReadAll(response.Body)
 	if err != nil {
 		return err
@@ -90,6 +97,12 @@ func (c *RawClient) Post(ctx context.Context, endpoint string, v any, result any
 	}
 	defer response.Body.Close()
 
+	// Check for HTTP errors
+	if response.StatusCode >= 400 {
+		buf, _ := io.ReadAll(response.Body)
+		return fmt.Errorf("HTTP %d: %s", response.StatusCode, string(buf))
+	}
+
 	if result == nil {
 		_, err := io.Copy(io.Discard, response.Body)
 		return err
@@ -123,6 +136,12 @@ func (c *RawClient) Delete(ctx context.Context, endpoint string) error {
 	}
 	defer response.Body.Close()
 
+	// Check for HTTP errors
+	if response.StatusCode >= 400 {
+		buf, _ := io.ReadAll(response.Body)
+		return fmt.Errorf("HTTP %d: %s", response.StatusCode, string(buf))
+	}
+
 	_, err = io.Copy(io.Discard, response.Body)
 	return err
 }

cmd/docker-mcp/oauth/auth.go

@@ -3,6 +3,7 @@ package oauth
 import (
 	"context"
 	"fmt"
+	"strings"
 
 	"github.com/docker/mcp-gateway/cmd/docker-mcp/internal/desktop"
 )
@@ -12,9 +13,18 @@ func Authorize(ctx context.Context, app string, scopes string) error {
 
 	authResponse, err := client.PostOAuthApp(ctx, app, scopes, false)
 	if err != nil {
+		// Check if error is about provider not found
+		if strings.Contains(err.Error(), "not found") {
+			return fmt.Errorf("OAuth provider does not exist")
+		}
 		return err
 	}
 
+	// Check if the response contains a valid browser URL
+	if authResponse.BrowserURL == "" {
+		return fmt.Errorf("OAuth provider does not exist")
+	}
+
 	fmt.Printf("Opening your browser for authentication. If it doesn't open automatically, please visit: %s\n", authResponse.BrowserURL)
 
 	return nil