zitadel / zitadel

ZITADEL - Identity infrastructure, simplified for you.
https://zitadel.com
Apache License 2.0
8.77k stars 551 forks source link

High number of "could not create sms channel" in the logs when SMS provider is not configured #7739

Open mffap opened 6 months ago

mffap commented 6 months ago

Preflight Checklist

Describe your problem

Given there's no SMS provider configured, when actions are initiated that would use the notification channel SMS, then ZITADEL writes the following error message in the logs

time="2024-04-09T08:46:21Z" level=error msg="could not create sms channel" caller="/home/runner/work/zitadel/zitadel/internal/notification/types/user_phone.go:24" error="ID=QUERY-fn99w Message=Errors.SMSConfig.NotExisting Parent=(sql: no rows in result set)"

This leads to a lot of noise / spamming of the logs (which was reported by our customers). Additionally this would also mean that some functionality fails for end-users, that could be with error code or in silent.

Describe your ideal solution

Multiple solutions or combinations are possible.

Version

No response

Environment

Self-hosted

Additional Context

Reported by customer

stebenz commented 4 months ago

Created a summarized issue https://github.com/zitadel/zitadel/issues/8042, will move this into the backlog.