Merge pull request #4259 from communitybridge/fix/gh_repo_autoenable_…

…issue fix auto enable github issue
communitybridge · Feb 27, 2024 · f552e1c · f552e1c
2 parents 2f29b52 + af22493
commit f552e1c
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 7 deletions.
diff --git a/cla-backend-go/cmd/server.go b/cla-backend-go/cmd/server.go
@@ -367,6 +367,11 @@ func server(localMode bool) http.Handler {
 	sign.Configure(v2API, v2SignService, usersService)
 	v2GithubActivity.Configure(v2API, v2GithubActivityService)
 
+	v2API.AddMiddlewareFor("POST", "/signed/individual/{installation_id}/{github_repository_id}/{change_request_id}", sign.DocusignMiddleware)
+	v2API.AddMiddlewareFor("POST", "/signed/corporate/{project_id}/{company_id}", sign.CCLADocusignMiddleware)
+	v2API.AddMiddlewareFor("POST", "/signed/gitlab/individual/{user_id}/{organization_id}/{gitlab_repository_id}/{merge_request_id}", sign.DocusignMiddleware)
+	v2API.AddMiddlewareFor("POST", "/signed/gerrit/individual/{user_id}", sign.DocusignMiddleware)
+
 	userCreaterMiddleware := func(next http.Handler) http.Handler {
 		return http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
 			createUserFromRequest(authorizer, usersService, eventsService, r)

diff --git a/cla-backend-go/v2/sign/handlers.go b/cla-backend-go/v2/sign/handlers.go
@@ -32,8 +32,8 @@ var (
 	cclaDocusignPayload []byte
 )
 
-// docusignMiddleware is used to get access to xml request body
-func docusignMiddleware(next http.Handler) http.Handler {
+// DocusignMiddleware is used to get access to xml request body
+func DocusignMiddleware(next http.Handler) http.Handler {
 	return http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
 		f := logrus.Fields{
 			"functionName": "v2.sign.handlers.docusignMiddleware",
@@ -53,7 +53,8 @@ func docusignMiddleware(next http.Handler) http.Handler {
 	})
 }
 
-func cclaDocusignMiddleware(next http.Handler) http.Handler {
+// CCLADocusignMiddleware used to set CCLA middleware
+func CCLADocusignMiddleware(next http.Handler) http.Handler {
 	return http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
 		f := logrus.Fields{
 			"functionName": "v2.sign.handlers.cclaDocusignMiddleware",
@@ -243,10 +244,6 @@ func Configure(api *operations.EasyclaAPI, service Service, userService users.Se
 			return sign.NewCclaCallbackOK()
 		})
 
-	api.AddMiddlewareFor("POST", "/signed/individual/{installation_id}/{github_repository_id}/{change_request_id}", docusignMiddleware)
-	api.AddMiddlewareFor("POST", "/signed/corporate/{project_id}/{company_id}", cclaDocusignMiddleware)
-	api.AddMiddlewareFor("POST", "/signed/gitlab/individual/{user_id}/{organization_id}/{gitlab_repository_id}/{merge_request_id}", docusignMiddleware)
-	api.AddMiddlewareFor("POST", "/signed/gerrit/individual/{user_id}", docusignMiddleware)
 }
 
 type codedResponse interface {