On Sun, Feb 07, 2021 at 05:42:44PM +0800, Luan Shengde wrote:
[why] for forwarded email from security admins, the email content will contain my_name/my_email(s) waiting to be assigned an account. a forwarded email may contain multi my_name/my_email(s), just loop them.
add store lab to ES
Signed-off-by: Luan Shengde shdluan@163.com
container/mail-robot/lib/apply-account.rb | 49 ++++++++++++++++++++--- 1 file changed, 43 insertions(+), 6 deletions(-)
diff --git a/container/mail-robot/lib/apply-account.rb b/container/mail-robot/lib/apply-account.rb index e8288d7..bc340ad 100755 --- a/container/mail-robot/lib/apply-account.rb +++ b/container/mail-robot/lib/apply-account.rb @@ -54,6 +54,8 @@ class ApplyAccount @mail_content = mail_content @es_host = @send_mail_host @es_port = ES_PORT
- # email address file for security admins.
- @sec_admins = YAML.load_file '/etc/compass-ci/conf/sec_admins.yaml'
Well who's this? It's not something readible and maintainable.
- 123456789@qq.com
Better change to such format
123456789@qq.com: name: xxx org: xxx project: xxx
where name/org/project is information for human maintenance.
Thanks, Fengguang