mirror of
https://gh.wpcy.net/https://github.com/discourse/discourse.git
synced 2026-05-22 02:34:05 +08:00
b4085c7b50
We didn't have support in the #problem constructor for multiple targets, forcing developers to manually construct a Problem instance. This involves a lot of details and is error prone. This PR supports passing an optional target argument to the constructor. This will be passed on to the translation_data method to generate the correct message as well.
75 lines
2 KiB
Ruby
Vendored
75 lines
2 KiB
Ruby
Vendored
# frozen_string_literal: true
|
|
|
|
##
|
|
# If group SMTP or IMAP has been configured, we want to make sure the
|
|
# credentials are always valid otherwise emails will not be sending out
|
|
# from group inboxes. This check is run as part of scheduled admin
|
|
# problem checks, and if any credentials have issues they will show up on
|
|
# the admin dashboard as a high priority issue.
|
|
class ProblemCheck::GroupEmailCredentials < ProblemCheck
|
|
self.priority = "high"
|
|
self.perform_every = 30.minutes
|
|
|
|
def call
|
|
[*smtp_errors, *imap_errors]
|
|
end
|
|
|
|
private
|
|
|
|
def targets
|
|
[*Group.with_smtp_configured.pluck(:name), *Group.with_imap_configured.pluck(:name)]
|
|
end
|
|
|
|
def translation_data(group)
|
|
{ group_name: group.name, group_full_name: group.full_name }
|
|
end
|
|
|
|
def smtp_errors
|
|
return [] if !SiteSetting.enable_smtp
|
|
|
|
Group.with_smtp_configured.find_each.filter_map do |group|
|
|
try_validate(group) do
|
|
EmailSettingsValidator.validate_smtp(
|
|
host: group.smtp_server,
|
|
port: group.smtp_port,
|
|
username: group.email_username,
|
|
password: group.email_password,
|
|
)
|
|
end
|
|
end
|
|
end
|
|
|
|
def imap_errors
|
|
return [] if !SiteSetting.enable_imap
|
|
|
|
Group.with_imap_configured.find_each.filter_map do |group|
|
|
try_validate(group) do
|
|
EmailSettingsValidator.validate_imap(
|
|
host: group.imap_server,
|
|
port: group.imap_port,
|
|
username: group.email_username,
|
|
password: group.email_password,
|
|
)
|
|
end
|
|
end
|
|
end
|
|
|
|
def try_validate(group, &blk)
|
|
begin
|
|
blk.call
|
|
nil
|
|
rescue *EmailSettingsExceptionHandler::EXPECTED_EXCEPTIONS => err
|
|
error_message =
|
|
EmailSettingsExceptionHandler.friendly_exception_message(err, group.smtp_server)
|
|
|
|
problem(group, override_data: { error: error_message })
|
|
rescue => err
|
|
Discourse.warn_exception(
|
|
err,
|
|
message:
|
|
"Unexpected error when checking SMTP credentials for group #{group.id} (#{group.name}).",
|
|
)
|
|
nil
|
|
end
|
|
end
|
|
end
|
