Skip to content

Commit

Permalink
Use logger instead of fmt.Println in brwoser.go
Browse files Browse the repository at this point in the history
`fmt.Println` still logs when `--quiet` is specified and that's problematic when saml2login is being used as credential_process in aws profiles.
  • Loading branch information
faridnsh authored and Farid Nouri Neshat committed Sep 4, 2023
1 parent 7494170 commit cbc1c78
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions pkg/provider/browser/browser.go
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@ package browser

import (
"errors"
"fmt"
"net/url"
"regexp"

Expand Down Expand Up @@ -94,7 +93,7 @@ var getSAMLResponse = func(page playwright.Page, loginDetails *creds.LoginDetail
return "", err
}

fmt.Println("waiting ...")
logger.Info("waiting ...")
r, _ := page.WaitForRequest(signin_re)
data, err := r.PostData()
if err != nil {
Expand Down

0 comments on commit cbc1c78

Please sign in to comment.