Merge pull request #6520 from devtron-labs/webhook-changes

fix: added more fields in webhook notification

Author: prkhrkat

.gitignore

@@ -4,3 +4,5 @@
 .env
 /cmd/external-app/devtron-ea
 devtron
+
+.qodo

pkg/notifier/WebhookNotificationService.go

@@ -109,6 +109,8 @@ func (impl *WebhookNotificationServiceImpl) GetWebhookVariables() (map[string]be
 		"devtronCiPipelineId":       beans.DevtronCiPipelineId,
 		"devtronCdPipelineId":       beans.DevtronCdPipelineId,
 		"devtronTriggeredByEmail":   beans.DevtronTriggeredByEmail,
+		"devtronPipelineType":       beans.DevtronPipelineType,
+		"devtronBuildGitCommitHash": beans.DevtronBuildGitCommitHash,
 		"eventType":                 beans.EventType,
 	}
 
@@ -119,7 +121,7 @@ func (impl *WebhookNotificationServiceImpl) FetchAllWebhookNotificationConfig()
 	var responseDto []*beans.WebhookConfigDto
 	webhookConfigs, err := impl.webhookRepository.FindAll()
 	if err != nil && !util.IsErrNoRows(err) {
-		impl.logger.Errorw("cannot find all webhoook config", "err", err)
+		impl.logger.Errorw("cannot find all webhook config", "err", err)
 		return []*beans.WebhookConfigDto{}, err
 	}
 	for _, webhookConfig := range webhookConfigs {

pkg/notifier/beans/beans.go

@@ -23,6 +23,8 @@ const (
 	DevtronCiPipelineId       WebhookVariable = "{{devtronCiPipelineId}}"
 	DevtronCdPipelineId       WebhookVariable = "{{devtronCdPipelineId}}"
 	DevtronTriggeredByEmail   WebhookVariable = "{{devtronTriggeredByEmail}}"
+	DevtronBuildGitCommitHash WebhookVariable = "{{devtronBuildGitCommitHash}}"
+	DevtronPipelineType       WebhookVariable = "{{devtronPipelineType}}"
 	EventType                 WebhookVariable = "{{eventType}}"
 )