Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FAQ] [Alarm instance management] user must be m #3125

Closed
3 tasks done
niuhu3 opened this issue Feb 1, 2024 · 1 comment
Closed
3 tasks done

[FAQ] [Alarm instance management] user must be m #3125

niuhu3 opened this issue Feb 1, 2024 · 1 comment
Labels
FAQ Frequently Asked Questions

Comments

@niuhu3
Copy link

niuhu3 commented Feb 1, 2024

Search before asking

  • I had searched in the issues and found no similar issues.

What happened

我设置了一个邮箱报警实例,测试的时候报错User must not be null,经过排查发现是前端没有传user参数,后端有这个判断,所以报错

What you expected to happen

Snipaste_2024-02-01_14-01-06

How to reproduce

Snipaste_2024-02-01_14-01-06

前端参数:

{id: 1, name: "email", type: "Email",…}
enabled
:
true
id
:
1
name
:
"email"
params
:
"{"receivers":"[email protected]","receiverCcs":"[email protected]","serverHost":"hadoop100","serverPort":"[email protected]","sender":"二狗子","msgtype":"text"}"
type
:
"Email"

后端校验: MailSender.java
mailUser = config.get(EmailConstants.NAME_MAIL_USER);
requireNonNull(mailUser, EmailConstants.NAME_MAIL_USER + mustNotNull);

Anything else

no

Version

0.7.5

Are you willing to submit PR?

  • Yes I am willing to submit a PR!

Code of Conduct

@niuhu3 niuhu3 added Bug Something isn't working Waiting for reply Waiting for reply labels Feb 1, 2024
@github-actions github-actions bot changed the title [Bug] [报警实例管理] user must be m [Bug] [Alarm instance management] user must be m Feb 1, 2024
@Zzm0809 Zzm0809 removed the Waiting for reply Waiting for reply label Feb 1, 2024
@Zzm0809
Copy link
Contributor

Zzm0809 commented Feb 1, 2024

你不开认证怎么发送呢 , 开启邮箱验证 填账号和密码

@Zzm0809 Zzm0809 added FAQ Frequently Asked Questions and removed Bug Something isn't working labels Feb 1, 2024
@Zzm0809 Zzm0809 changed the title [Bug] [Alarm instance management] user must be m [FAQ] [Alarm instance management] user must be m Feb 1, 2024
@Zzm0809 Zzm0809 closed this as completed Feb 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FAQ Frequently Asked Questions
Projects
None yet
Development

No branches or pull requests

2 participants