Skip to content

Commit 2d742d0

Browse files
Remove V1 corporate console
Signed-off-by: Lukasz Gryglicki <[email protected]> Assisted by [OpenAI](https://platform.openai.com/) Assisted by [GitHub Copilot](https://github.com/features/copilot)
1 parent f8af0dd commit 2d742d0

File tree

8 files changed

+21
-20
lines changed

8 files changed

+21
-20
lines changed

cla-backend-go/cmd/dynamo_events_lambda/main.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -138,7 +138,7 @@ func init() {
138138
v2RepositoryService := v2Repositories.NewService(repositoriesRepo, v2Repository, projectClaGroupRepo, githubOrganizationsRepo, gitlabOrganizationRepo, eventsService)
139139
gitlabOrgService := gitlab_organizations.NewService(gitlabOrganizationRepo, v2RepositoryService, projectClaGroupRepo, storeRepo, usersService, signaturesRepo, companyRepo)
140140

141-
companyService := company.NewService(companyRepo, configFile.CorporateConsoleV1URL, userRepo, usersService)
141+
companyService := company.NewService(companyRepo, configFile.CorporateConsoleV2URL, userRepo, usersService)
142142
v2CompanyService := v2Company.NewService(companyService, signaturesRepo, projectRepo, usersRepo, companyRepo, projectClaGroupRepo, eventsService)
143143
organization_service.InitClient(configFile.APIGatewayURL, eventsService)
144144
acs_service.InitClient(configFile.APIGatewayURL, configFile.AcsAPIKey)

cla-backend-go/cmd/server.go

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -316,10 +316,10 @@ func server(localMode bool) http.Handler {
316316
healthService := health.New(Version, Commit, Branch, BuildDate)
317317
templateService := template.NewService(stage, templateRepo, docraptorClient, awsSession)
318318
v1ProjectService := service.NewService(v1CLAGroupRepo, gitV1Repository, gerritRepo, v1ProjectClaGroupRepo, usersRepo)
319-
emailTemplateService := emails.NewEmailTemplateService(v1CLAGroupRepo, v1ProjectClaGroupRepo, v1ProjectService, configFile.CorporateConsoleV1URL, configFile.CorporateConsoleV2URL)
319+
emailTemplateService := emails.NewEmailTemplateService(v1CLAGroupRepo, v1ProjectClaGroupRepo, v1ProjectService, configFile.CorporateConsoleV2URL, configFile.CorporateConsoleV2URL)
320320
emailService := emails.NewService(emailTemplateService, v1ProjectService)
321321
v2ProjectService := v2Project.NewService(v1ProjectService, v1CLAGroupRepo, v1ProjectClaGroupRepo)
322-
v1CompanyService := v1Company.NewService(v1CompanyRepo, configFile.CorporateConsoleV1URL, userRepo, usersService)
322+
v1CompanyService := v1Company.NewService(v1CompanyRepo, configFile.CorporateConsoleV2URL, userRepo, usersService)
323323
v2CompanyService := v2Company.NewService(v1CompanyService, signaturesRepo, v1CLAGroupRepo, usersRepo, v1CompanyRepo, v1ProjectClaGroupRepo, eventsService)
324324
v2CurrentUserService := v2CurrentUser.NewService()
325325

@@ -329,7 +329,7 @@ func server(localMode bool) http.Handler {
329329
gitlabOrganizationsService := gitlab_organizations.NewService(gitlabOrganizationRepo, v2RepositoriesService, v1ProjectClaGroupRepo, storeRepository, usersService, signaturesRepo, v1CompanyRepo)
330330
v1SignaturesService := signatures.NewService(signaturesRepo, v1CompanyService, usersService, eventsService, githubOrgValidation, v1RepositoriesService, githubOrganizationsService, v1ProjectService, gitlabApp, configFile.ClaV1ApiURL, configFile.CLALandingPage, configFile.CLALogoURL)
331331
v2SignatureService := v2Signatures.NewService(awsSession, configFile.SignatureFilesBucket, v1ProjectService, v1CompanyService, v1SignaturesService, v1ProjectClaGroupRepo, signaturesRepo, usersService, approvalsRepo)
332-
v1ClaManagerService := cla_manager.NewService(claManagerReqRepo, v1ProjectClaGroupRepo, v1CompanyService, v1ProjectService, usersService, v1SignaturesService, eventsService, emailTemplateService, configFile.CorporateConsoleV1URL)
332+
v1ClaManagerService := cla_manager.NewService(claManagerReqRepo, v1ProjectClaGroupRepo, v1CompanyService, v1ProjectService, usersService, v1SignaturesService, eventsService, emailTemplateService, configFile.CorporateConsoleV2URL)
333333
v2ClaManagerService := v2ClaManager.NewService(emailTemplateService, v1CompanyService, v1ProjectService, v1ClaManagerService, usersService, v1RepositoriesService, v2CompanyService, eventsService, v1ProjectClaGroupRepo)
334334
v1ApprovalListService := approval_list.NewService(approvalListRepo, v1ProjectClaGroupRepo, v1ProjectService, usersRepo, v1CompanyRepo, v1CLAGroupRepo, signaturesRepo, emailTemplateService, configFile.CorporateConsoleV2URL, http.DefaultClient)
335335
authorizer := auth.NewAuthorizer(authValidator, userRepo)
@@ -383,7 +383,7 @@ func server(localMode bool) http.Handler {
383383
v2Repositories.Configure(v2API, v2RepositoriesService, eventsService)
384384
gerrits.Configure(api, gerritService, v1ProjectService, eventsService)
385385
v2Gerrits.Configure(v2API, gerritService, v1ProjectService, eventsService, v1ProjectClaGroupRepo)
386-
v2Company.Configure(v2API, v2CompanyService, v1ProjectClaGroupRepo, configFile.LFXPortalURL, configFile.CorporateConsoleV1URL)
386+
v2Company.Configure(v2API, v2CompanyService, v1ProjectClaGroupRepo, configFile.LFXPortalURL)
387387
v2CurrentUser.Configure(v2API, v2CurrentUserService)
388388
cla_manager.Configure(api, v1ClaManagerService, v1CompanyService, v1ProjectService, usersService, v1SignaturesService, eventsService, emailTemplateService)
389389
v2ClaManager.Configure(v2API, v2ClaManagerService, v1CompanyService, configFile.LFXPortalURL, configFile.CorporateConsoleV2URL, v1ProjectClaGroupRepo, userRepo)

cla-backend-go/config/config.go

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,6 @@ type Config struct {
6464
AllowedOrigins []string `json:"-"`
6565

6666
CorporateConsoleURL string `json:"corporateConsoleURL"`
67-
CorporateConsoleV1URL string `json:"corporateConsoleV1URL"`
6867
CorporateConsoleV2URL string `json:"corporateConsoleV2URL"`
6968

7069
CLAContributorv2Base string `json:"cla-contributor-v2-base"`

cla-backend-go/config/ssm.go

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,7 @@ func loadSSMConfig(awsSession *session.Session, stage string) Config { //nolint
7676
fmt.Sprintf("cla-gitlab-app-redirect-uri-%s", stage),
7777
fmt.Sprintf("cla-gitlab-app-web-hook-uri-%s", stage),
7878
fmt.Sprintf("cla-corporate-base-%s", stage),
79-
fmt.Sprintf("cla-corporate-v1-base-%s", stage),
79+
8080
fmt.Sprintf("cla-corporate-v2-base-%s", stage),
8181
fmt.Sprintf("cla-contributor-v2-base-%s", stage),
8282
fmt.Sprintf("cla-doc-raptor-api-key-%s", stage),
@@ -187,8 +187,7 @@ func loadSSMConfig(awsSession *session.Session, stage string) Config { //nolint
187187
config.CLALogoURL = resp.value
188188
case fmt.Sprintf("cla-corporate-base-%s", stage):
189189
config.CorporateConsoleURL = resp.value
190-
case fmt.Sprintf("cla-corporate-v1-base-%s", stage):
191-
config.CorporateConsoleV1URL = resp.value
190+
192191
case fmt.Sprintf("cla-corporate-v2-base-%s", stage):
193192
config.CorporateConsoleV2URL = resp.value
194193
case fmt.Sprintf("cla-doc-raptor-api-key-%s", stage):

cla-backend-go/utils/email.go

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -101,10 +101,8 @@ func SendEmail(subject string, body string, recipients []string) error {
101101

102102
// GetCorporateURL returns the corporate URL based on the specified flag
103103
func GetCorporateURL(isV2Project bool) string {
104-
if isV2Project {
105-
return config.GetConfig().CorporateConsoleV2URL
106-
}
107-
return config.GetConfig().CorporateConsoleV1URL
104+
// V1 corporate console has been shut down - always return V2 URL
105+
return config.GetConfig().CorporateConsoleV2URL
108106
}
109107

110108
// GetEmailHelpContent returns the standard email help paragraph details.

cla-backend-go/v2/company/handlers.go

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ import (
2727
)
2828

2929
// Configure sets up the middleware handlers
30-
func Configure(api *operations.EasyclaAPI, service Service, projectClaGroupRepo projects_cla_groups.Repository, LFXPortalURL, v1CorporateConsole string) { // nolint
30+
func Configure(api *operations.EasyclaAPI, service Service, projectClaGroupRepo projects_cla_groups.Repository, LFXPortalURL string) { // nolint
3131

3232
api.CompanyGetCompanyByInternalIDHandler = company.GetCompanyByInternalIDHandlerFunc(
3333
func(params company.GetCompanyByInternalIDParams, authUser *auth.User) middleware.Responder {
@@ -637,11 +637,15 @@ func Configure(api *operations.EasyclaAPI, service Service, projectClaGroupRepo
637637
ctx := context.WithValue(context.Background(), utils.XREQUESTID, reqID) // nolint
638638

639639
corporateLink := ""
640-
// Get appropirate corporate link (v1|v2)
640+
// Get appropriate corporate link - V1 is no longer supported, redirect to V2
641641
if params.Body.Version == "v1" {
642-
corporateLink = v1CorporateConsole
642+
// V1 corporate console has been shut down - redirect to V2
643+
corporateLink = LFXPortalURL
643644
} else if params.Body.Version == "v2" {
644645
corporateLink = LFXPortalURL
646+
} else {
647+
// Default to V2 if version is not specified
648+
corporateLink = LFXPortalURL
645649
}
646650

647651
err := service.RequestCompanyAdmin(ctx, params.UserID, params.Body.ClaManagerEmail.String(), params.Body.ClaManagerName, params.Body.ContributorName, params.Body.ContributorEmail.String(), params.Body.ProjectName, params.Body.CompanyName, corporateLink)

cla-backend/cla/routes.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1275,7 +1275,7 @@ def request_corporate_signature(
12751275
{
12761276
"project_id": "d8cead54-92b7-48c5-a2c8-b1e295e8f7f1",
12771277
"company_id": "83f61e34-4457-45a6-a7ab-449ad6efcfbb",
1278-
"return_url": "https://corporate.v1.easycla.lfx.linuxfoundation.org/#/company/83f61e34-4457-45a6-a7ab-449ad6efcfbb"
1278+
"return_url": "https://organization.lfx.linuxfoundation.org/#/company/83f61e34-4457-45a6-a7ab-449ad6efcfbb"
12791279
}
12801280
12811281
Creates a new signature given project and company IDs. The manager will be redirected to the

cla-backend/cla/utils.py

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1927,10 +1927,11 @@ def get_email_sign_off_content() -> str:
19271927
def get_corporate_url(project_version: str) -> str:
19281928
"""
19291929
helper method that returns appropriate corporate link based on EasyCLA version
1930-
:param project_version: cla_group version(v1|v2)
1931-
:return: default is v1 corporate console
1930+
:param project_version: cla_group version(v1|v2) - deprecated, always returns V2
1931+
:return: V2 corporate console (V1 has been shut down)
19321932
"""
1933-
return CORPORATE_V2_BASE if project_version == "v2" else CORPORATE_BASE
1933+
# V1 corporate console has been shut down - always return V2
1934+
return CORPORATE_V2_BASE
19341935

19351936

19361937
def append_email_help_sign_off_content(body: str, project_version: str) -> str:

0 commit comments

Comments
 (0)