Skip to content

[Security][High] Redact Communication remoteParty in webhook payloads #107

Description

@BechsteinDigital

Severity

High

Problem

Communication call events serialize the remote telephone number under remoteParty. The plugin manifest declares no sensitive fields for this payload, and the default sensitive-field registry does not include remoteParty. Consequently, webhooks can transmit the number even when IncludeSensitiveData=false.

The documentation describes other field names and therefore does not protect the actual emitted schema.

Evidence

  • custom/static-plugins/Communication/src/Application/Calls/CallBusinessEvent.cs:65
  • custom/static-plugins/Communication/registry.json:1
  • src/Core/Application/Webhooks/SensitivePayloadFieldRegistry.cs:15
  • src/Core/Application/Webhooks/WebhookDispatcher.cs:60
  • docs-site/guides/fundamentals/registry-manifest.md:121

Acceptance criteria

  • The actual call-event schema declares every telephone-number field as sensitive.
  • Default webhook delivery masks remoteParty.
  • Sensitive delivery remains possible only through the explicit privileged setting.
  • A test dispatches a real CallBusinessEvent through the production minimizer.
  • Schema and documentation use the same field names.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions