Merge pull request #13 from EikaGruppen/only-scope-param-if-set

fix: only send scope param if set

Author: tormodatt

authorize.go

@@ -49,7 +49,9 @@ func authorizeUrl(opts Options, port int) (request authorizeRequest, err error)
 		"code_challenge":        {codeChallange},
 		"code_challenge_method": {"S256"},
 		"state":                 {request.state},
-		"scope":                 {strings.Join(opts.Scopes, " ")},
+	}
+	if len(opts.Scopes) > 0 {
+		q.Add("scope", strings.Join(opts.Scopes, " "))
 	}
 
 	for k, v := range opts.AuthorizationExtParams {

go.mod

@@ -1,3 +1,3 @@
 module github.com/EikaGruppen/go-oauth-cli-client
 
-go 1.20
+go 1.24

oauth.go

@@ -4,7 +4,7 @@ import (
 	"encoding/json"
 	"errors"
 	"fmt"
-	"io/ioutil"
+	"io"
 	"net"
 	"net/http"
 	"net/url"
@@ -71,7 +71,7 @@ func getAuthorizationCode(opts Options, code string, authorizeRequest authorizeR
 	}
 	defer response.Body.Close()
 
-	body, err := ioutil.ReadAll(response.Body)
+	body, err := io.ReadAll(response.Body)
 	if err != nil {
 		return nil, err
 	}

refresh.go

@@ -3,7 +3,7 @@ package oauth
 import (
 	"encoding/json"
 	"fmt"
-	"io/ioutil"
+	"io"
 	"net/http"
 	"net/url"
 )
@@ -21,7 +21,7 @@ func Refresh(opts Options, refreshToken string) (*TokenResponse, error) {
 	}
 	defer response.Body.Close()
 
-	body, err := ioutil.ReadAll(response.Body)
+	body, err := io.ReadAll(response.Body)
 	if err != nil {
 		return nil, err
 	}

revoke.go

@@ -3,7 +3,7 @@ package oauth
 import (
 	"encoding/json"
 	"fmt"
-	"io/ioutil"
+	"io"
 	"net/http"
 	"net/url"
 )
@@ -37,7 +37,7 @@ func Revoke(opts Options, tokenType TokenType, token string) error {
 	}
 	defer response.Body.Close()
 
-	body, err := ioutil.ReadAll(response.Body)
+	body, err := io.ReadAll(response.Body)
 	if err != nil {
 		return err
 	}