BaryoDev / BaryoDev/barakoCMS

Connectors: record outbound request deliveries the way WebhookAction does

Open
#671 0 comments 0 reactions 0 assignees View on GitHub
core enhancement
Dominant language
C#
Stars
6
Forks
7
Avg merge
4h 42m
Merged PRs (30d)
307

Description

Found while planning barakocms.com as a barakoCMS-backed site.

`WebhookAction` writes a `WebhookDelivery` row for every attempt, with the response and timing, and `/api/webhook-deliveries` serves them. `RequestAction` sends through `ConnectorSender`, which returns a status and elapsed time and records nothing beyond the workflow run.

## Why this matters

The two actions do the same thing, an outbound HTTP call from a workflow, with different observability. "Did the request fire, and what came back" is answerable for webhooks and not for connector requests. When a request action fails intermittently, the workflow run says the step failed and the response body is gone.

The retention service (`WebhookDeliveryRetentionService`) already exists, so the storage and cleanup side is solved.

## What to change

Record outbound connector requests the same way, subject to the same retention, with the credential redacted the way `WithoutSecret` does for parameters.

Contributor guide

Open the contributing guide

Research direction

Start by comparing WebhookAction and its WebhookDelivery records with RequestAction and ConnectorSender, which returns the status and elapsed time. Read WebhookDeliveryRetentionService and WithoutSecret to understand the existing retention and credential-redaction conventions. Done means connector outbound attempts retain the relevant response and timing information under the same cleanup rules without exposing credentials.

Written by the indexing model from the issue text.

Assessment

Tech stack
csharp
Domain
api, backend
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Active
Clarity
Mostly clear
Newbie friendliness
64/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.