Monorepo for Tangled tangled.org

appview/notify: add Push event and webhook notifier #1067

merged opened by anirudh.fi targeting master from icy/qlyxxp

Add Push method to Notifier interface for git push events. Implement WebhookNotifier that sends webhook payloads with HMAC-SHA256 signatures for authentication. Supports push events with delivery tracking and retry logic (3 attempts with exponential backoff).

Signed-off-by: Anirudh Oppiliappan anirudh@tangled.org

Labels

None yet.

assignee

None yet.

Participants 2
AT URI
at://did:plc:hwevmowznbiukdf6uk5dwrrq/sh.tangled.repo.pull/3menyebs7qn22
Interdiff #2 #3
appview/notify/db/db.go

This file has not been changed.

appview/notify/logging_notifier.go

This file has not been changed.

appview/notify/merged_notifier.go

This file has not been changed.

appview/notify/notifier.go

This file has not been changed.

appview/notify/webhook_notifier.go

This file has not been changed.

History

6 rounds 3 comments
sign up or login to add to the discussion
1 commit
expand
appview/notify: add Push event and webhook notifier
3/3 success
expand
expand 0 comments
pull request successfully merged
1 commit
expand
appview/notify: add Push event and webhook notifier
3/3 success
expand
expand 0 comments
1 commit
expand
appview/notify: add Push event and webhook notifier
3/3 success
expand
expand 0 comments
1 commit
expand
appview/notify: add Push event and webhook notifier
3/3 success
expand
expand 0 comments
1 commit
expand
appview/notify: add Push event and webhook notifier
3/3 success
expand
expand 3 comments
  • would be nice to have a strongly typed payload object here
  • the value for user-agent seems a bit strange, its Tangled-Hook/<short-sha>, would it be better to identify the repo here?

changeset lgtm otherwise!

GitHub does the same for their hook version. I suppose we could have a separate header with the repository info?

makes sense, let's do the same then

1 commit
expand
appview/notify: add Push event and webhook notifier
3/3 success
expand
expand 0 comments