summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJeroen Wijenbergh <jeroen.wijenbergh@geant.org>2026-02-12 14:20:23 +0100
committerJeroen Wijenbergh <jeroen.wijenbergh@geant.org>2026-02-12 14:31:05 +0100
commit3593e449a5abcc1a33173e75cdd8155763b23be8 (patch)
treebf120d67ad2899c7350ac599ac8ccd27dd4719fc
parenta20ccbd7ad758495e80ad180fb12f874a45e4ae2 (diff)
eduvpnapi: simplify OAuth post bool
-rw-r--r--internal/eduvpnapi/eduvpnapi.go5
-rw-r--r--internal/loglevel/loglevel.go2
2 files changed, 2 insertions, 5 deletions
diff --git a/internal/eduvpnapi/eduvpnapi.go b/internal/eduvpnapi/eduvpnapi.go
index 62fe0d1..a045865 100644
--- a/internal/eduvpnapi/eduvpnapi.go
+++ b/internal/eduvpnapi/eduvpnapi.go
@@ -75,11 +75,8 @@ func NewAPI(ctx context.Context, clientID string, sd ServerData, cb Callbacks, t
// Construct OAuth
transp := sd.Transport()
- post := true
// we do not support non-loopback clients with response_mode form_post
- if cr != "" {
- post = false
- }
+ post := cr == ""
o := eduoauth.OAuth{
ClientID: clientID,
EndpointFunc: func(ctx context.Context) (*eduoauth.EndpointResponse, error) {
diff --git a/internal/loglevel/loglevel.go b/internal/loglevel/loglevel.go
index b851e3a..9be6178 100644
--- a/internal/loglevel/loglevel.go
+++ b/internal/loglevel/loglevel.go
@@ -35,7 +35,7 @@ func (l *Logger) Init(dir string) (*slog.Logger, error) {
return slog.New(handler), nil
}
-// Closes closes, meaning cleans up the logger file
+// Close cleans up the logger, meaning closes the logger file
func (l *Logger) Close() error {
return l.fr.Close()
}