Merge pull request #14 from Feandil/sprintfFormatFix

glpatcern · web-flow · commit da1f18ff8f48 · 2025-02-07T08:11:09.000+01:00
Fix missing escaping for '%' in fmt.Sprintf.
diff --git a/user/rest.go b/user/rest.go
@@ -196,7 +196,7 @@ func (i *Identity) UserType() userpb.UserType {
 }
 
 func (m *manager) fetchAllUserAccounts(ctx context.Context) error {
-	url := fmt.Sprintf("%s/api/v1.0/Identity?filter=unconfirmed%3Afalse&field=upn&field=primaryAccountEmail&field=displayName&field=uid&field=gid&field=type&field=source&field=activeUser", m.conf.APIBaseURL)
+	url := fmt.Sprintf("%s/api/v1.0/Identity?filter=unconfirmed%%3Afalse&field=upn&field=primaryAccountEmail&field=displayName&field=uid&field=gid&field=type&field=source&field=activeUser", m.conf.APIBaseURL)
 
 	for {
 		var r IdentitiesResponse

Original file line number	Diff line number	Diff line change
`@@ -196,7 +196,7 @@ func (i *Identity) UserType() userpb.UserType {`
`196`	`196`	`}`
`197`	`197`
`198`	`198`	`func (m *manager) fetchAllUserAccounts(ctx context.Context) error {`
`199`		`- url := fmt.Sprintf("%s/api/v1.0/Identity?filter=unconfirmed%3Afalse&field=upn&field=primaryAccountEmail&field=displayName&field=uid&field=gid&field=type&field=source&field=activeUser", m.conf.APIBaseURL)`
	`199`	`+ url := fmt.Sprintf("%s/api/v1.0/Identity?filter=unconfirmed%%3Afalse&field=upn&field=primaryAccountEmail&field=displayName&field=uid&field=gid&field=type&field=source&field=activeUser", m.conf.APIBaseURL)`
`200`	`200`
`201`	`201`	`for {`
`202`	`202`	`var r IdentitiesResponse`