Skip to content

Handle webhook inputs consistently and in one place #421

Closed

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

Activity

  1. added
    debtCode quality improvement or decrease of technical debt.
    t-toolingIssues with this label are in the ownership of the tooling team.
    on Jun 16, 2025
  2. shubhamos-ai commented on Aug 29, 2025

    @shubhamos-ai

    I'm interested in contributing to this project! The issue description is clear and I have some ideas for implementation. Would love to discuss the approach.

  3. added this to the 3.0 milestone on Feb 26, 2026
  4. self-assigned this
    on Feb 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

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

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions