Merge pull request #4500 from nickmango/feature/gerrit-redirect

nickmango · web-flow · commit c99d6eee69f6 · 2024-12-12T18:23:04.000+03:00
Bug/Gerrit Return URL
diff --git a/cla-backend-go/v2/sign/service.go b/cla-backend-go/v2/sign/service.go
@@ -2343,8 +2343,8 @@ func (s *service) RequestIndividualSignatureGerrit(ctx context.Context, input *m
 		return nil, errors.New("no gerrits found for project")
 	}
 
-	returnURL := gerrits.List[0].GerritURL
-	log.WithFields(f).Debugf("returnURL: %s", returnURL)
+	returnURL := input.ReturnURL
+	log.WithFields(f).Debugf("returnURL: %s", input.ReturnURL)
 
 	if latestSignature != nil {
 		log.WithFields(f).Debugf("comparing latest signature document version: %s to latest document version: %s", latestSignature.SignatureDocumentMajorVersion, latestDocument.DocumentMajorVersion)

Original file line number	Diff line number	Diff line change
`@@ -2343,8 +2343,8 @@ func (s service) RequestIndividualSignatureGerrit(ctx context.Context, input m`
`2343`	`2343`	`return nil, errors.New("no gerrits found for project")`
`2344`	`2344`	`}`
`2345`	`2345`
`2346`		`- returnURL := gerrits.List[0].GerritURL`
`2347`		`- log.WithFields(f).Debugf("returnURL: %s", returnURL)`
	`2346`	`+ returnURL := input.ReturnURL`
	`2347`	`+ log.WithFields(f).Debugf("returnURL: %s", input.ReturnURL)`
`2348`	`2348`
`2349`	`2349`	`if latestSignature != nil {`
`2350`	`2350`	`log.WithFields(f).Debugf("comparing latest signature document version: %s to latest document version: %s", latestSignature.SignatureDocumentMajorVersion, latestDocument.DocumentMajorVersion)`