Merge pull request #6 from stephanpieterse/master

dnitsch · web-flow · commit d923d42aaa58 · 2023-10-25T10:21:55.000+01:00
Additional split on SAMLResponse split
diff --git a/internal/web/web.go b/internal/web/web.go
@@ -75,6 +75,7 @@ func (web *Web) GetSamlLogin(conf config.SamlConfig) (string, error) {
 	wait()
 
 	saml := strings.Split(page.MustElement(`body`).MustText(), "SAMLResponse=")[1]
+	saml = strings.Split(saml, "&")[0]
 	return nurl.QueryUnescape(saml)
 
 }
@@ -95,7 +96,7 @@ func (web *Web) ClearCache() error {
 	return nil
 }
 
-//checkRodProcess gets a list running process
+// checkRodProcess gets a list running process
 // kills any hanging rod browser process from any previous improprely closed sessions
 func checkRodProcess() error {
 	pids := make([]int, 0)

Original file line number	Diff line number	Diff line change
`@@ -75,6 +75,7 @@ func (web *Web) GetSamlLogin(conf config.SamlConfig) (string, error) {`
`75`	`75`	`wait()`
`76`	`76`
`77`	`77`	saml := strings.Split(page.MustElement(`body`).MustText(), "SAMLResponse=")[1]
	`78`	`+ saml = strings.Split(saml, "&")[0]`
`78`	`79`	`return nurl.QueryUnescape(saml)`
`79`	`80`
`80`	`81`	`}`
`@@ -95,7 +96,7 @@ func (web *Web) ClearCache() error {`
`95`	`96`	`return nil`
`96`	`97`	`}`
`97`	`98`
`98`		`-//checkRodProcess gets a list running process`
	`99`	`+// checkRodProcess gets a list running process`
`99`	`100`	`// kills any hanging rod browser process from any previous improprely closed sessions`
`100`	`101`	`func checkRodProcess() error {`
`101`	`102`	`pids := make([]int, 0)`