Skip to content

Handle webhook inputs consistently and in one place #421

Open
@Pijukatel

Description

@Pijukatel

Unify Webhook input sanitation and handling. Maybe with Pydantic?

Context:
Webhook inputs to the clients are now handled in different ways based on the client used. One path is using encode_webhook_list_to_base64, for example in ActorClient.start.

The other one is using get_webhook_representation, for example in WebhookClient.update.

This creates situation where same responsibility is handled in two different places and it can happen than one of the paths has issues that are already fixed in the other one. For example: #419

Metadata

Metadata

Assignees

No one assigned

    Labels

    debtCode quality improvement or decrease of technical debt.t-toolingIssues with this label are in the ownership of the tooling team.

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions