Use the browser package

RiotGames · Oct 4, 2023 · 65558d6 · 65558d6
1 parent 0869d42
commit 65558d6
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 23 deletions.
diff --git a/cli/browser_darwin.go b/cli/browser_darwin.go
diff --git a/cli/browser_linux.go b/cli/browser_linux.go
diff --git a/cli/get.go b/cli/get.go
@@ -12,6 +12,7 @@ import (
 	"github.com/aws/aws-sdk-go/aws"
 	"github.com/aws/aws-sdk-go/aws/session"
 	"github.com/aws/aws-sdk-go/service/sts"
+	"github.com/pkg/browser"
 	"github.com/riotgames/key-conjurer/internal/api"
 	"github.com/spf13/cobra"
 )
@@ -302,7 +303,7 @@ func (p OktaTencentCloudSAMLProvider) FetchSAMLAssertion(ctx context.Context) ([
 
 	// TODO: We should probably wait a second before opening the browser to see if the http server is going to throw an error
 	// If it throws an error, we should bail
-	if err := OpenBrowser(p.Href); err != nil {
+	if err := browser.OpenURL(p.Href); err != nil {
 		return nil, fmt.Errorf("failed to open web browser to URL %s: %w", p.Href, err)
 	}