This is not needed according to AI

lukaszgryglicki · lukaszgryglicki · commit ff8366aa7fd0 · 2026-04-29T09:56:27.000+02:00
Signed-off-by: Lukasz Gryglicki <lgryglicki@cncf.io> Assisted by [OpenAI](https://platform.openai.com/) Assisted by [GitHub Copilot](https://github.com/features/copilot)
diff --git a/cla-backend-legacy/internal/api/handlers.go b/cla-backend-legacy/internal/api/handlers.go
@@ -9538,7 +9538,7 @@ func (h *Handlers) GetReturnUrlV2(w http.ResponseWriter, r *http.Request) {
 	}
 
 	if returnURL := getAttrString(sig, "signature_return_url"); returnURL != "" {
-		h.refreshGitHubChangeRequestBeforeReturn(ctx, signatureID, sig)
+		// h.refreshGitHubChangeRequestBeforeReturn(ctx, signatureID, sig)
 		// Legacy Python (cla/controllers/signing.py::return_url) has an eventual-consistency wait loop
 		// for v2 company signatures: it checks that all CLA managers listed in signature_acl have
 		// the "cla-manager" role assigned (via platform org service scopes) before redirecting.

Original file line number	Diff line number	Diff line change
`@@ -9538,7 +9538,7 @@ func (h Handlers) GetReturnUrlV2(w http.ResponseWriter, r http.Request) {`
`9538`	`9538`	`}`
`9539`	`9539`
`9540`	`9540`	`if returnURL := getAttrString(sig, "signature_return_url"); returnURL != "" {`
`9541`		`- h.refreshGitHubChangeRequestBeforeReturn(ctx, signatureID, sig)`
	`9541`	`+ // h.refreshGitHubChangeRequestBeforeReturn(ctx, signatureID, sig)`
`9542`	`9542`	`// Legacy Python (cla/controllers/signing.py::return_url) has an eventual-consistency wait loop`
`9543`	`9543`	`// for v2 company signatures: it checks that all CLA managers listed in signature_acl have`
`9544`	`9544`	`// the "cla-manager" role assigned (via platform org service scopes) before redirecting.`