Merge pull request #68 from actiontech/issuse-1024

fix: testing webhook availability
actiontech · Oct 23, 2023 · e9fac51 · e9fac51
2 parents 5a0f04c + 130b841
commit e9fac51
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/internal/dms/biz/notify_config.go b/internal/dms/biz/notify_config.go
@@ -440,8 +440,8 @@ func (d *WebHookConfigurationUsecase) TestWebHookConfiguration(ctx context.Conte
 	if !webhookC.Enable {
 		return fmt.Errorf("webhook notice is not enabled")
 	}
-	// TODO 测试webhook配置可用性
-	return err
+
+	return d.webhookSendRequest(ctx, "hello")
 }
 
 func (d *WebHookConfigurationUsecase) SendWebHookMessage(ctx context.Context, triggerEventType string /*TODO validate TriggerEventType*/, message string) error {