From 03619ed2287ca106106881c2f9e15634a3d47ae2 Mon Sep 17 00:00:00 2001
From: Drew DeVault <sir@cmpwn.com>
Date: Sat, 10 Oct 2020 11:31:24 -0400
Subject: [PATCH] webhooks/legacy: update only affected delivery

Forgot to add this where clause, whoops!
---
 webhooks/legacy.go      | 1 +
 webhooks/legacy_test.go | 2 +-
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/webhooks/legacy.go b/webhooks/legacy.go
index 4e431e1f75d8b7705d080316f5a7acf6ade58e18..d76cf79287277597a495284e7485201a6bb39d32 100644
--- a/webhooks/legacy.go
+++ b/webhooks/legacy.go
@@ -230,6 +230,7 @@ func deliverPayload(ctx context.Context, name, url string,
 			Set("response", string(body)).
 			Set("response_status", resp.StatusCode).
 			Set("response_headers", sb.String()).
+			Where("id = ?", deliveryID).
 			RunWith(tx).
 			ExecContext(ctx)
 		return err
diff --git a/webhooks/legacy_test.go b/webhooks/legacy_test.go
index 26cb27276840eb90c5f4fa3e9d083841c40ee580..2b08c0583354e893bf496514026bece3077c4104 100644
--- a/webhooks/legacy_test.go
+++ b/webhooks/legacy_test.go
@@ -103,7 +103,7 @@ func TestDelivery(t *testing.T) {
 
 	mock.ExpectBegin()
 	mock.ExpectExec(`UPDATE user_webhook_delivery`).
-		WithArgs("Thanks!", 200, sqlmock.AnyArg()) // Any => response headers
+		WithArgs("Thanks!", 200, sqlmock.AnyArg(), 4096) // Any => response headers
 	mock.ExpectCommit()
 
 	ctx = database.Context(context.Background(), db)