From a779b11a80536d17c6097a8169f009d6597a078e Mon Sep 17 00:00:00 2001 From: Aleksandr Nogikh Date: Mon, 7 Nov 2022 09:59:43 +0000 Subject: dashboard: temporarily introduce the HandleListEmails option It will help us gradually deploy the feature. --- dashboard/app/app_test.go | 6 ++++-- dashboard/app/reporting_email.go | 5 +++++ 2 files changed, 9 insertions(+), 2 deletions(-) diff --git a/dashboard/app/app_test.go b/dashboard/app/app_test.go index 3ab0b72e0..87baeaae4 100644 --- a/dashboard/app/app_test.go +++ b/dashboard/app/app_test.go @@ -253,14 +253,16 @@ var testConfig = &GlobalConfig{ Name: "access-user-reporting1", DailyLimit: 1000, Config: &EmailConfig{ - Email: "test@syzkaller.com", + Email: "test@syzkaller.com", + HandleListEmails: true, }, }, { Name: "access-public-reporting2", DailyLimit: 1000, Config: &EmailConfig{ - Email: "test2@syzkaller.com", + Email: "test2@syzkaller.com", + HandleListEmails: true, }, }, }, diff --git a/dashboard/app/reporting_email.go b/dashboard/app/reporting_email.go index 63381acc2..ea25721b3 100644 --- a/dashboard/app/reporting_email.go +++ b/dashboard/app/reporting_email.go @@ -65,6 +65,7 @@ var mailingLists map[string]bool type EmailConfig struct { Email string + HandleListEmails bool // This is a temporary option to simplify the feature deployment. MailMaintainers bool DefaultMaintainers []string SubjectPrefix string @@ -512,6 +513,10 @@ func matchBugFromList(c context.Context, sender, subject string) (*bugInfoResult log.Infof(c, "reporting is not EmailConfig (%q)", subject) continue } + if !emailConfig.HandleListEmails { + log.Infof(c, "the feature is disabled for the config") + continue + } if emailConfig.Email != sender { log.Infof(c, "config's Email is %v, wanted %v", emailConfig.Email, sender) continue -- cgit mrf-deployment