-
Notifications
You must be signed in to change notification settings - Fork 161
Description
Is your feature request related to a problem? Please describe.
It's frustrating that Convoy currently uses a GET
request to validate an endpoint during configuration or testing. This can cause unintended side effects on user endpoints, especially if GET
has side effects or triggers logic on the receiving service.
Describe the solution you'd like
Convoy should issue an OPTIONS
request instead of a GET
when validating an endpoint. Alternatively, we could simply dial the endpoint to check connectivity (e.g., using a HEAD
request or TCP dial) without sending an HTTP method that could be misinterpreted as a meaningful API call.
Describe alternatives you've considered
- Continuing with
GET
, but warning users about the side effects (not ideal). - Allowing the HTTP method for validation to be configurable.
- Using
HEAD
, although not all servers support it correctly. - Performing a raw TCP dial to validate connectivity.
Additional context
Using OPTIONS
is a common and safe way to probe endpoints without triggering business logic. Some webhook providers (like GitHub) follow this pattern for validation. This change would improve developer experience and reduce the risk of unexpected behavior during setup.