Skip to content

280 webhook secret#285

Merged
supun-io merged 9 commits intomainfrom
280-webhook-secret
Oct 13, 2025
Merged

280 webhook secret#285
supun-io merged 9 commits intomainfrom
280-webhook-secret

Conversation

@Nolab0
Copy link
Copy Markdown
Member

@Nolab0 Nolab0 commented Oct 2, 2025

No description provided.

@Nolab0 Nolab0 requested a review from supun-io October 2, 2025 10:36
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Oct 10, 2025

Codecov Report

❌ Patch coverage is 87.50000% with 4 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
worker/webhooks.go 66.66% 3 Missing and 1 partial ⚠️

📢 Thoughts on this report? Let us know!

@Nolab0
Copy link
Copy Markdown
Member Author

Nolab0 commented Oct 10, 2025

#280

Copy link
Copy Markdown
Member

@supun-io supun-io left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unlike API keys, users should be able to copy the webhook secret later.

  • Do not send the key in the WebhookObject
  • Add a button in the WebhookRow to copy the secret. This calls a new API endpoint which returns the secret. Then, copies to the clipboard
  • Remove the modal that shows "this is the only time...."

Comment thread backend/src/Api/Console/Object/WebhookObject.php Outdated
Comment thread backend/src/Entity/WebhookDelivery.php Outdated
Comment thread backend/migrations/Version20250624093228.php Outdated
Comment thread worker/webhooks_test.go Outdated
@supun-io supun-io merged commit 69cacd2 into main Oct 13, 2025
3 of 5 checks passed
@supun-io supun-io deleted the 280-webhook-secret branch October 13, 2025 08:41
supun-io added a commit that referenced this pull request Oct 24, 2025
* Add secret key to backend

* Fix test

* Fix test

* Add signature in DB and go

* test fixes

* minor updates

* Change front to copy secret

* Fix doc

---------

Co-authored-by: Supun Wimalasena <supun@hyvor.com>
Co-authored-by: Supun Wimalasena <44988673+supun-io@users.noreply.github.com>
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.

3 participants