Merge pull request #177 from linuxfoundation/feat/LFXV2-3094-chat-webhook-url

[LFXV2-3094] feat(committee): add chat_webhook_url field to Committee domain model

Author: andrest50

cmd/committee-api/design/committee.go

@@ -35,6 +35,7 @@ var _ = dsl.Service("committee-service", func() {
 			CommitteeBaseAttributes()
 
 			CommitteeSettingsAttributes()
+			ChatWebhookURLAttribute()
 
 			WritersAttribute()
 			AuditorsAttribute()
@@ -228,6 +229,7 @@ var _ = dsl.Service("committee-service", func() {
 
 			CommitteeUIDAttribute()
 			CommitteeSettingsAttributes()
+			ChatWebhookURLAttribute()
 
 			WritersAttribute()
 			AuditorsAttribute()

cmd/committee-api/design/type.go

@@ -208,6 +208,12 @@ func DescriptionAttribute() {
 // would otherwise pass dsl.FormatURI as syntactically valid URIs.
 const urlPattern = `^https?://[^\s/$.?#][^\s]*$`
 
+// httpsURLPattern validates an HTTPS-only URL or an empty string.
+// Used for write-only credential fields (e.g. webhook URLs) where:
+//   - a non-empty value must be a valid HTTPS URL (plaintext HTTP is rejected)
+//   - an empty string is the explicit clear signal (see ChatWebhookURLAttribute)
+const httpsURLPattern = `^$|^https://[^\s/$.?#][^\s]*$`
+
 // WebsiteAttribute is the DSL attribute for committee website.
 func WebsiteAttribute() {
 	dsl.Attribute("website", dsl.String, "The website URL of the committee", func() {
@@ -1028,6 +1034,17 @@ func ChatChannelAttribute() {
 	})
 }
 
+// ChatWebhookURLAttribute is the DSL attribute for the committee's Slack Incoming Webhook URL.
+// This field is write-only: accepted on create and PUT settings, never returned from GET.
+// Update semantics: omit or send null to preserve the stored URL; send "" to clear it.
+func ChatWebhookURLAttribute() {
+	dsl.Attribute("chat_webhook_url", dsl.String, "Slack Incoming Webhook URL for sharing content to a Slack channel. Write-only: never returned from GET. Send empty string to clear a previously stored value; omit the field (or send null) to preserve the existing value.", func() {
+		dsl.Pattern(httpsURLPattern)
+		dsl.MaxLength(500)
+		dsl.Example("https://hooks.slack.example.org/services/TXXXXXXXX/BXXXXXXXX/placeholder")
+	})
+}
+
 // InviteUIDAttribute is the DSL attribute for invite UID in URL paths.
 func InviteUIDAttribute() {
 	dsl.Attribute("invite_uid", dsl.String, "Committee invite UID", func() {

cmd/committee-api/service/committee_service_response.go

@@ -87,6 +87,7 @@ func (s *committeeServicesrvc) convertPayloadToBase(p *committeeservice.CreateCo
 func (s *committeeServicesrvc) convertPayloadToSettings(p *committeeservice.CreateCommitteePayload) *model.CommitteeSettings {
 	settings := &model.CommitteeSettings{
 		BusinessEmailRequired: p.BusinessEmailRequired,
+		ChatWebhookURL:        p.ChatWebhookURL,
 		LastReviewedBy:        p.LastReviewedBy,
 		Writers:               convertPayloadUsersToModel(p.Writers, nil),
 		Auditors:              convertPayloadUsersToModel(p.Auditors, nil),
@@ -295,6 +296,7 @@ func (s *committeeServicesrvc) convertPayloadToUpdateSettings(p *committeeservic
 	settings := &model.CommitteeSettings{
 		UID:                   *p.UID, // UID is required for updates
 		BusinessEmailRequired: p.BusinessEmailRequired,
+		ChatWebhookURL:        p.ChatWebhookURL,
 		LastReviewedAt:        p.LastReviewedAt,
 		LastReviewedBy:        p.LastReviewedBy,
 		Writers:               convertPayloadUsersToModel(p.Writers, existingWriters),