Skip to content

Comments

Fix/api webhook validation cleanup#628

Closed
Dobrunia wants to merge 8 commits intomasterfrom
fix/api-webhook-validation-cleanup
Closed

Fix/api webhook validation cleanup#628
Dobrunia wants to merge 8 commits intomasterfrom
fix/api-webhook-validation-cleanup

Conversation

@Dobrunia
Copy link
Member

Description:

  • Allow saving enabled webhook with empty endpoint (valid intermediate state) — validate only when endpoint is non-empty

…erty to NotificationsChannelsDBScheme interface
… and related tests. Integrate private IP validation into ipValidator module for improved code organization.
…mproved error handling. Update validation logic to await results from channel checks, enhancing overall reliability.
… ALLOWED_PORTS constants from ipValidator module, improving code organization and maintainability.
…o remove endpoint checks when isEnabled is true, streamlining the validation process.
@Dobrunia Dobrunia closed this Feb 18, 2026
@Dobrunia Dobrunia deleted the fix/api-webhook-validation-cleanup branch February 18, 2026 16:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant