Sync main with dev.

.github/workflows/deploy-dev.yml

@@ -164,7 +164,7 @@ jobs:
           echo "Validating v2 backend using endpoint: ${v2_url}"
           curl --fail -XGET ${v2_url}
           exit_code=$?
-          if [[ ${exit_coe} -eq 0 ]]; then
+          if [[ ${exit_code} -eq 0 ]]; then
             echo "Successful response from endpoint: ${v2_url}"
           else
             echo "Failed to get a successful response from endpoint: ${v2_url}"
@@ -174,7 +174,7 @@ jobs:
           echo "Validating v3 backend using endpoint: ${v3_url}"
           curl --fail -XGET ${v3_url}
           exit_code=$?
-          if [[ ${exit_coe} -eq 0 ]]; then
+          if [[ ${exit_code} -eq 0 ]]; then
             echo "Successful response from endpoint: ${v3_url}"
             # JSON response should include "Status": "healthy"
             if [[ `curl -s -XGET ${v3_url} | jq -r '.Status'` == "healthy" ]]; then
@@ -207,7 +207,7 @@ jobs:
           echo "Validating v4 backend using endpoint: ${v4_url}"
           curl --fail -XGET ${v4_url}
           exit_code=$?
-          if [[ ${exit_coe} -eq 0 ]]; then
+          if [[ ${exit_code} -eq 0 ]]; then
             echo "Successful response from endpoint: ${v4_url}"
             # JSON response should include "Status": "healthy"
             if [[ `curl -s -XGET ${v4_url} | jq -r '.Status'` == "healthy" ]]; then

.github/workflows/deploy-prod.yml

@@ -130,7 +130,7 @@ jobs:
           echo "Validating v2 backend using endpoint: ${v2_url}"
           curl --fail -XGET ${v2_url}
           exit_code=$?
-          if [[ ${exit_coe} -eq 0 ]]; then
+          if [[ ${exit_code} -eq 0 ]]; then
             echo "Successful response from endpoint: ${v2_url}"
           else
             echo "Failed to get a successful response from endpoint: ${v2_url}"
@@ -140,7 +140,7 @@ jobs:
           echo "Validating v3 backend using endpoint: ${v3_url}"
           curl --fail -XGET ${v3_url}
           exit_code=$?
-          if [[ ${exit_coe} -eq 0 ]]; then
+          if [[ ${exit_code} -eq 0 ]]; then
             echo "Successful response from endpoint: ${v3_url}"
             # JSON response should include "Status": "healthy"
             if [[ `curl -s -XGET ${v3_url} | jq -r '.Status'` == "healthy" ]]; then
@@ -172,7 +172,7 @@ jobs:
           echo "Validating v4 backend using endpoint: ${v4_url}"
           curl --fail -XGET ${v4_url}
           exit_code=$?
-          if [[ ${exit_coe} -eq 0 ]]; then
+          if [[ ${exit_code} -eq 0 ]]; then
             echo "Successful response from endpoint: ${v4_url}"
             # JSON response should include "Status": "healthy"
             if [[ `curl -s -XGET ${v4_url} | jq -r '.Status'` == "healthy" ]]; then

.gitignore

@@ -258,3 +258,4 @@ cover.out
 src.txt
 src.txt.*
 out.json
+body.json

BOT_ALLOWLIST.md

@@ -146,6 +146,7 @@ To add first `skip_cla` value for an organization:
 ```
 aws --profile lfproduct-prod --region us-east-1 dynamodb update-item --table-name "cla-prod-github-orgs" --key '{"organization_name": {"S": "open-telemetry"}}' --update-expression 'SET skip_cla = :val' --expression-attribute-values '{":val": {"M": {"otel-arrow":{"S":";re:^\\d+\\+Copilot@users\\.noreply\\.github\\.com$;copilot-swe-agent[bot]"}}}}'
 aws --profile lfproduct-prod --region us-east-1 dynamodb update-item --table-name "cla-prod-github-orgs" --key '{"organization_name": {"S": "openfga"}}' --update-expression 'SET skip_cla = :val' --expression-attribute-values '{":val": {"M": {"vscode-ext":{"S":";re:^\\d+\\+Copilot@users\\.noreply\\.github\\.com$;copilot-swe-agent[bot]"}}}}'
+aws --profile lfproduct-prod --region us-east-1 dynamodb update-item --table-name "cla-prod-github-orgs" --key '{"organization_name": {"S": "eslint"}}' --update-expression 'SET skip_cla = :val' --expression-attribute-values '{":val": {"M": {"*":{"S":"[Copilot;re:^\\d+\\+Copilot@users\\.noreply\\.github\\.com$;copilot-swe-agent[bot]||;re:^\\d+\\+Copilot@users\\.noreply\\.github\\.com$;copilot-swe-agent[bot]||Copilot;re:^\\d+\\+copilot@users\\.noreply\\.github\\.com$;Copilot]"}}}}'
 ```
 
 To add additional repositories entries without overwriting the existing `skip_cla` value:

cla-backend-go/swagger/cla.v2.yaml

@@ -351,6 +351,7 @@ paths:
         - $ref: "#/parameters/x-email"
         - name: claGroupInput
           in: body
+          required: true
           schema:
             $ref: '#/definitions/create-cla-group-input'
       responses:
@@ -2663,6 +2664,7 @@ paths:
         - $ref: "#/parameters/path-userID"
         - name: body
           in: body
+          required: true
           schema:
             type: object
             required:
@@ -2829,6 +2831,7 @@ paths:
         - $ref: "#/parameters/path-companyID"
         - $ref: "#/parameters/path-projectSFID"
         - name: body
+          required: true
           in: body
           schema:
             type: object
@@ -2998,6 +3001,7 @@ paths:
         - $ref: "#/parameters/path-companyID"
         - name: body
           in: body
+          required: true
           schema:
             type: object
             x-nullable: false
@@ -3043,6 +3047,7 @@ paths:
         - $ref: "#/parameters/path-companyID"
         - name: body
           in: body
+          required: true
           schema:
             type: object
             x-nullable: false
@@ -4637,7 +4642,9 @@ parameters:
     description: the Salesforce ID of the Foundation
     in: query
     type: string
-    pattern: '^[a-zA-Z0-9]{18}|[a-zA-Z0-9]{15}$' # see: https://stackoverflow.com/questions/9742913/validating-a-salesforce-id
+    pattern: '^([0-9A-Za-z]{15}|[0-9A-Za-z]{18})$' # see: https://stackoverflow.com/questions/9742913/validating-a-salesforce-id
+    minLength: 15
+    maxLength: 18
   companyID:
     name: companyID
     description: The internal company ID representing signing entity name instance (EasyCLA)
@@ -4658,18 +4665,18 @@ parameters:
     in: path
     type: string
     required: true
-    pattern: '^[a-zA-Z0-9]{18}|[a-zA-Z0-9]{15}$' # see: https://stackoverflow.com/questions/9742913/validating-a-salesforce-id
-    minLength: 5
-    maxLength: 255
+    pattern: '^([0-9A-Za-z]{15}|[0-9A-Za-z]{18})$' # see: https://stackoverflow.com/questions/9742913/validating-a-salesforce-id
+    minLength: 15
+    maxLength: 18
   path-projectSFID:
     name: projectSFID
     description: the Salesforce ID of the project
     in: path
     type: string
     required: true
-    pattern: '^[a-zA-Z0-9]{18}|[a-zA-Z0-9]{15}$' # see: https://stackoverflow.com/questions/9742913/validating-a-salesforce-id
-    minLength: 5
-    maxLength: 255
+    pattern: '^([0-9A-Za-z]{15}|[0-9A-Za-z]{18})$' # see: https://stackoverflow.com/questions/9742913/validating-a-salesforce-id
+    minLength: 15
+    maxLength: 18
   path-userID:
     name: userID
     description: v1 EasyCLA user Admin ID
@@ -4699,7 +4706,9 @@ parameters:
     in: path
     type: string
     required: true
-    pattern: '^[a-zA-Z0-9]{18}|[a-zA-Z0-9]{15}$' # see: https://stackoverflow.com/questions/9742913/validating-a-salesforce-id
+    pattern: '^([0-9A-Za-z]{15}|[0-9A-Za-z]{18})$' # see: https://stackoverflow.com/questions/9742913/validating-a-salesforce-id
+    minLength: 15
+    maxLength: 18
   path-companyName:
     name: companyName
     description: the company name

cla-backend-go/swagger/common/properties/external-id.yaml

@@ -3,4 +3,6 @@
 
 type: string
 example: "00117000015vpjXAAQ"
-pattern: '^[a-zA-Z0-9]{18}|[a-zA-Z0-9]{15}$' # see: https://stackoverflow.com/questions/9742913/validating-a-salesforce-id
+pattern: '^([0-9A-Za-z]{15}|[0-9A-Za-z]{18})$' # see: https://stackoverflow.com/questions/9742913/validating-a-salesforce-id
+minLength: 15
+maxLength: 18

cla-backend-go/v2/cla_manager/service.go

@@ -1167,8 +1167,14 @@ func (s *service) NotifyCLAManagers(ctx context.Context, notifyCLAManagers *mode
 	// Search for Easy CLA User
 	log.WithFields(f).Debugf("Getting user by ID: %s", notifyCLAManagers.UserID)
 	userModel, userErr := s.easyCLAUserService.GetUser(notifyCLAManagers.UserID)
+	// log.WithFields(f).Debugf("Got user by ID: %s -> (%+v, %+v)", notifyCLAManagers.UserID, userModel, userErr)
 	if userErr != nil {
-		msg := fmt.Sprintf("Problem getting user by ID: %s ", notifyCLAManagers.UserID)
+		msg := fmt.Sprintf("Problem getting user by ID: %s: %v", notifyCLAManagers.UserID, userErr)
+		log.WithFields(f).Warn(msg)
+		return ErrCLAUserNotFound
+	}
+	if userModel == nil {
+		msg := fmt.Sprintf("Problem getting user by ID: %s", notifyCLAManagers.UserID)
 		log.WithFields(f).Warn(msg)
 		return ErrCLAUserNotFound
 	}

cla-backend-go/v2/signatures/service.go

@@ -381,6 +381,10 @@ func (s *Service) InvalidateICLA(ctx context.Context, claGroupID string, userID
 		log.WithFields(f).Debug("unable to get user record")
 		return userErr
 	}
+	if user == nil {
+		log.WithFields(f).Debug("unable to get user record")
+		return fmt.Errorf("user not found for userID: %s", userID)
+	}
 
 	log.WithFields(f).Debug("invalidating signature record ...")
 	note := fmt.Sprintf("Signature invalidated (approved set to false) by %s for %s ", authUser.UserName, utils.GetBestUsername(user))

cla-backend/requirements.txt

@@ -56,3 +56,4 @@ wrapt==1.17.2
 astroid==3.3.8
 pluggy==1.5.0
 gunicorn==22.0.0
+PyNaCl==1.5.0

tests/functional/README.md

@@ -90,6 +90,10 @@ You can ask for example `.env` file over slack.
 - Run `npx cypress install`
 - Run tests using cmd `npx cypress run`. Or `xvfb-run -a npx cypress run` when runnign over SSH.
 - Run tests using UI `npx cypress open`. Choose **E2E testing**, select **Chrome** browser.
+- Run single E2E spec file: `` [ALLOW_FAIL=1] ./utils/run-single-test.sh github-repositories ``.
+- Run single test from E2E spec file: `` [ALLOW_FAIL=1] ./utils/run-single-test.sh github-repositories 'test name regexp' ``.
+- Additional options include: `TOKEN=xxxx`, `XACL=xxxx`, `DEBUG=1`, `LOCAL=1`, `ALL=1`.
+- To format E2E tests: `` npx prettier --write cypress/e2e/* ``.
 - View test reports in the `cypress-report` directory.
 - Explore source code files for detailed implementation.
 

tests/functional/cypress.config.ts

@@ -9,6 +9,23 @@ export default defineConfig({
   e2e: {
     // baseUrl: 'http://localhost:1234',
     specPattern: 'cypress/e2e/**/**/*.{js,jsx,ts,tsx}',
+    setupNodeEvents(on, config) {
+      on('task', {
+        log(message: string) {
+          if (process.env.DEBUG) {
+            console.log(message);
+          }
+          return null;
+        },
+        table(rows: any[]) {
+          if (process.env.DEBUG) {
+            console.table(rows);
+          }
+          return null;
+        },
+      });
+      return config;
+    },
   },
   env: {
     APP_URL: process.env.APP_URL,
@@ -19,6 +36,5 @@ export default defineConfig({
     AUTH0_CLIENT_SECRET: process.env.AUTH0_CLIENT_SECRET,
     AUTH0_CLIENT_ID: process.env.AUTH0_CLIENT_ID,
     CYPRESS_ENV: process.env.CYPRESS_ENV,
-  }
+  },
 });
-

tests/functional/cypress/appConfig/config.dev.ts

@@ -1,51 +1,66 @@
 // config.dev.js
-export const appConfig = {   
-    projectSFID: 'a09P000000DsCE5IAN',//project name: SUN
-
-    /*Variable for create cla group*/
-     foundationSFID:'a09P000000DsNGsIAN', //project name: easyAutom foundation & common for foundation.spec.ts
-     createNewClaGroupSFID:'a09P000000DsNGxIAN',  //project name: easyAutom-child1
-     claGroupName:'CypressDevClaGroup',
-     gitHubOrgPartialStatus:'Sun-lfxfoundationOrgTest',//For partial Connection 
-     enrollProjectsSFID:'a09P000000DsNHCIA3', //project name: easyAutomChild1-GrandChild1,
-     child_Project_name:'easyAutomChild1-GrandChild1',
-
-    /*company*/
-     companyName:'Infosys Limited',
-     user_id:"8f3e52b8-0072-11ee-9def-0ef17207dfe8",//vthakur+lfstaff@contractor.linuxfoundation.org
-     userEmail: "vthakur+lfstaff@contractor.linuxfoundation.org",
-     user_id2: "4a4c1dba-407f-11ed-8c58-a6b0f8fb81a9",//vthakur+lfitstaff@contractor.linuxfoundation.org
-
-    /*events*/
-     companyID:"f7c7ac9c-4dbf-4104-ab3f-6b38a26d82dc",//infosys limited
-     companyExternalID:'0014100000Te0G7AAJ',//
-     childProjectSFID:'a09P000000DsNH2IAN', //project name: easyAutom-child2
-
-    /*github-organizations*/
-     gitHubOrgUpdate:'ApiAutomStandaloneOrg',
-     gitHubNewOrg:'cypressioTest',
-     claGroupId:'1baf67ab-d894-4edf-b6fc-c5f939db59f7',
-
-    /*metrics*/
-     projectID:'01af041c-fa69-4052-a23c-fb8c1d3bef24',
-
-    /*cla-manager*/
-     userIdclaManager:'c5ac2857-c263-11ed-94d1-d2349de32229',//veerendrat
-     claGroupId_projectSFID:'01af041c-fa69-4052-a23c-fb8c1d3bef24',//sun project claGroupID
-
-    /* project */
-     projectName:'easyAutom-child2',
-
-    /* GitLab-Organization*/
-     gitLabOrganizationName:'sunGitlabGroup',
-     gitLabOrganizationFullPath:'https://gitlab.com/cla_dev_automationgroup',
-     groupId:'72539854',
-
-     /* Signature */
-     lfid:'veerendrat',
-     emailApprovalList:'veerendrat@proximabiz.com',
-     gitOrgApprovalList:'proximabiz.com',
-     gitUsernameApprovalList:'vrthakur',
-     gitLabOrgApprovalList:'https://gitlab.com/veerendrat1',
-     domainApprovalList:'proximabiz.com',
-  };
+export const appConfig = {
+  projectSFID: 'a09P000000DsCE5IAN', //project name: SUN
+
+  /*Variable for create cla group*/
+  foundationSFID: 'a09P000000DsNGsIAN', //project name: easyAutom foundation & common for foundation.spec.ts
+  createNewClaGroupSFID: 'a09P000000DsNGxIAN', //project name: easyAutom-child1
+  claGroupName: 'CypressDevClaGroup',
+  gitHubOrgPartialStatus: 'Sun-lfxfoundationOrgTest', //For partial Connection
+  enrollProjectsSFID: 'a09P000000DsNHCIA3', //project name: easyAutomChild1-GrandChild1,
+  child_Project_name: 'easyAutomChild1-GrandChild1',
+
+  /*company*/
+  companyName: 'Infosys Limited',
+  user_id: '8f3e52b8-0072-11ee-9def-0ef17207dfe8', //vthakur+lfstaff@contractor.linuxfoundation.org
+  userEmail: 'vthakur+lfstaff@contractor.linuxfoundation.org',
+  user_id2: '4a4c1dba-407f-11ed-8c58-a6b0f8fb81a9', //vthakur+lfitstaff@contractor.linuxfoundation.org
+  user_id3: '9dcf5bbc-2492-11ed-97c7-3e2a23ea20b5', // lgryglicki
+  userEmail3: 'lukaszgryglicki@o2.pl',
+
+  /*events*/
+  companyID: 'f7c7ac9c-4dbf-4104-ab3f-6b38a26d82dc', //infosys limited
+  companyExternalID: '0014100000Te0G7AAJ', //
+  childProjectSFID: 'a09P000000DsNH2IAN', //project name: easyAutom-child2
+
+  /*github-organizations*/
+  gitHubOrgUpdate: 'ApiAutomStandaloneOrg',
+  gitHubNewOrg: 'cypressioTest',
+  claGroupId: '1baf67ab-d894-4edf-b6fc-c5f939db59f7',
+
+  /*metrics*/
+  projectID: '01af041c-fa69-4052-a23c-fb8c1d3bef24',
+
+  /*cla-manager*/
+  userIdclaManager: 'c5ac2857-c263-11ed-94d1-d2349de32229', //veerendrat
+  userIdclaManager2: '9dcf5bbc-2492-11ed-97c7-3e2a23ea20b5', // lgryglicki
+  claGroupId_projectSFID: '01af041c-fa69-4052-a23c-fb8c1d3bef24', //sun project claGroupID
+
+  /* project */
+  projectName: 'easyAutom-child2',
+  projectName2: 'child-2-cla-group',
+  projectSFID2: 'a092h000004wlQQAAY',
+  projectSFID3: 'a0941000002wBz4AAE',
+  projectID2: 'e48f8c9f-2d24-4d3f-9123-51eabe34bcc0',
+
+  /* GitLab-Organization*/
+  gitLabOrganizationName: 'sunGitlabGroup',
+  gitLabOrganizationFullPath: 'https://gitlab.com/cla_dev_automationgroup',
+  groupId: '72539854',
+
+  /* Signature */
+  lfid: 'veerendrat',
+  emailApprovalList: 'veerendrat@proximabiz.com',
+  gitOrgApprovalList: 'proximabiz.com',
+  gitUsernameApprovalList: 'vrthakur',
+  gitLabOrgApprovalList: 'https://gitlab.com/veerendrat1',
+  domainApprovalList: 'proximabiz.com',
+  // lgryglicki
+  lfid2: 'lgryglicki',
+  emailApprovalList2: 'lukaszgryglicki@o2.pl',
+  gitOrgApprovalList2: 'o2.pl',
+  gitUsernameApprovalList2: 'lukaszgryglicki',
+  gitLabOrgApprovalList2: 'https://gitlab.com/lukaszgryglicki',
+  domainApprovalList2: 'o2.pl',
+  lfid3: 'vthakur.github',
+};

tests/functional/cypress/appConfig/config.production.ts

@@ -1,8 +1,7 @@
 // config.production.js
-export const appConfig = {   
-    projectSFID: 'a09P000000DsCE5IAN',//project name: SUN
-    //Veriable used in foundation.spec.ts
-    foundationSFID:'a09P000000DsNGsIAN' //project name: easyAutom foundation
-    //Variable for create cla group
-
-  };
+export const appConfig = {
+  projectSFID: 'a09P000000DsCE5IAN', //project name: SUN
+  //Veriable used in foundation.spec.ts
+  foundationSFID: 'a09P000000DsNGsIAN', //project name: easyAutom foundation
+  //Variable for create cla group
+};