Skip to content

Commit

Permalink
Truncate and de-newline returnURL
Browse files Browse the repository at this point in the history
Limit the logged returnURL to just two lines and remove any
embedded newlines.

Signed-off-by: Kevin Lyda <kevin@lyda.ie>
  • Loading branch information
lyda committed Mar 28, 2024
1 parent 9b5f3e5 commit 9fa4582
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion util/oidc/oidc.go
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@ import (
gooidc "github.com/coreos/go-oidc/v3/oidc"
"github.com/golang-jwt/jwt/v4"
log "github.com/sirupsen/logrus"
"github.com/yuin/goldmark/text"
"golang.org/x/oauth2"

"github.com/argoproj/argo-cd/v2/common"
Expand Down Expand Up @@ -288,7 +289,8 @@ func (a *ClientApp) HandleLogin(w http.ResponseWriter, r *http.Request) {
returnURL := r.FormValue("return_url")
// Check if return_url is valid, otherwise abort processing (see https://github.com/argoproj/argo-cd/pull/4780)
if !isValidRedirectURL(returnURL, []string{a.settings.URL}) {
log.Errorf("Invalid redirect URL: %s is not a valid per the settings %s.", returnURL, a.settings.URL)
log.Errorf("Invalid redirect URL: %s is not a valid per the settings %s.",
strings.Replace(text.Trunc(returnURL, 160), "\n", "", -1), a.settings.URL)
http.Error(w, "Invalid redirect URL: the protocol and host (including port) must match and the path must be within allowed URLs if provided", http.StatusBadRequest)
return
}
Expand Down

0 comments on commit 9fa4582

Please sign in to comment.