Add webhook payload size optimization options #35129

kerwin612 · 2025-07-19T14:13:21Z

This PR adds webhook payload size optimization options to help reduce payload size for large commits or pushes with many commits. This feature is particularly useful for target URLs that restrict payload size or have performance problems handling large JSON parsing.

wxiaoguang · 2025-07-19T14:25:00Z

It should be more flexible to use "limit number" instead of "bool"

kerwin612 · 2025-07-23T08:56:26Z

It should be more flexible to use "limit number" instead of "bool"

Is this OK?

Signed-off-by: Kerwin Bryant <[email protected]>

wxiaoguang · 2025-07-30T11:16:42Z

IIRC all other places use "-1" for "no-limit" when "0" is used for "limit to zero".
- For example: the limit of the repo number
Maybe we need a JSON column but not many new separate columns.
There are some issues asking about "commit order", so some users want the first N commits, while some other users want the last N commits
- Use forward order for commits in webhook notifications #21313
- Don't reverse order of commits in webhooks #21252

kerwin612 · 2025-08-04T06:33:50Z

routers/api/v1/utils/hook.go

templates/repo/settings/webhook/settings.tmpl

web_src/js/features/repo-settings-webhook.ts

models/webhook/webhook.go

wxiaoguang · 2025-08-28T03:34:29Z

models/migrations/v1_25/v322.go

+
+func AddWebhookPayloadOptimizationColumns(x *xorm.Engine) error {
+	type Webhook struct {
+		MetaSettings string `xorm:"meta_settings TEXT"`


XORM natively supports JSON type IIRC

models/webhook/webhook.go

wxiaoguang · 2025-08-28T03:36:26Z

routers/api/v1/utils/hook.go

+}
+
+// getPayloadConfigLimit extracts the "limit" integer value from a payload config map
+func getPayloadConfigLimit(m map[string]any) int {


It seems fragile, I don't think we need these tricks to parse form input

wxiaoguang · 2025-08-28T03:37:55Z

tests/integration/repo_webhook_test.go

+					"files": map[string]any{
+						"enable": true,
+						"limit":  2,
+					},


What does "enable" mean?

If "enable=false", then "no files" or "no commits"? Or "enable=false" means "disable the limit"?

Co-authored-by: wxiaoguang <[email protected]> Signed-off-by: Kerwin Bryant <[email protected]>

…win612/gitea into kerwin612-feature/webhook-payload-optimization

Add webhook payload size optimization options

1915361

GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Jul 19, 2025

github-actions bot added modifies/translation modifies/go Pull requests that update Go code modifies/templates This PR modifies the template files modifies/migrations labels Jul 19, 2025

kerwin612 force-pushed the feature/webhook-payload-optimization branch 6 times, most recently from f249f67 to 47c395d Compare July 23, 2025 08:09

refactor webhook payload optimization: switch from bool to limit number

5472d66

kerwin612 force-pushed the feature/webhook-payload-optimization branch from 47c395d to 5472d66 Compare July 23, 2025 08:37

kerwin612 added 3 commits July 24, 2025 19:48

Merge branch 'main' into feature/webhook-payload-optimization

39718b2

Signed-off-by: Kerwin Bryant <[email protected]>

fix

9682610

refactor: swap webhook payload optimization logic (-1/0 values)

544e445

kerwin612 requested a review from lunny July 30, 2025 04:39

Merge branch 'main' into feature/webhook-payload-optimization

42a880f

lunny approved these changes Jul 30, 2025

View reviewed changes

GiteaBot added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Jul 30, 2025

add webhook payload optimization API support

6e07c6f

github-actions bot added the modifies/api This PR adds API routes or modifies them label Jul 30, 2025

Merge branch 'main' into feature/webhook-payload-optimization

11cdba1

kerwin612 force-pushed the feature/webhook-payload-optimization branch from 5e828e9 to 258d8ee Compare August 4, 2025 07:07

silverwind reviewed Aug 11, 2025

View reviewed changes

routers/api/v1/utils/hook.go Show resolved Hide resolved

silverwind reviewed Aug 11, 2025

View reviewed changes

templates/repo/settings/webhook/settings.tmpl Outdated Show resolved Hide resolved

kerwin612 added 2 commits August 15, 2025 14:30

Merge branch 'main' into feature/webhook-payload-optimization

6b635dc

clean code

6d932c8

github-actions bot added the modifies/frontend label Aug 15, 2025

clean code

eec07ee

kerwin612 requested a review from silverwind August 15, 2025 08:10

wxiaoguang reviewed Aug 15, 2025

View reviewed changes

web_src/js/features/repo-settings-webhook.ts Show resolved Hide resolved

wxiaoguang reviewed Aug 15, 2025

View reviewed changes

models/webhook/webhook.go Show resolved Hide resolved

wxiaoguang reviewed Aug 15, 2025

View reviewed changes

models/webhook/webhook.go Outdated Show resolved Hide resolved

kerwin612 added 6 commits August 24, 2025 16:40

Merge branch 'main' into feature/webhook-payload-optimization

80f8d2c

fix

a3581be

clean code

10d0450

clean code

f4ea1a1

clean code

98d4da1

clean code

94225ed

kerwin612 requested a review from wxiaoguang August 24, 2025 14:02

hiifong approved these changes Aug 28, 2025

View reviewed changes

GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Aug 28, 2025

wxiaoguang reviewed Aug 28, 2025

View reviewed changes

models/webhook/webhook.go Outdated Show resolved Hide resolved

wxiaoguang reviewed Aug 28, 2025

View reviewed changes

wxiaoguang marked this pull request as draft August 28, 2025 03:39

Update models/webhook/webhook.go

46c0824

Co-authored-by: wxiaoguang <[email protected]> Signed-off-by: Kerwin Bryant <[email protected]>

lunny modified the milestones: 1.25.0, 1.26.0 Sep 18, 2025

lunny added 2 commits September 17, 2025 17:59

Merge branch 'feature/webhook-payload-optimization' of github.com:ker…

7f9debb

…win612/gitea into kerwin612-feature/webhook-payload-optimization

revert unnecessary change

ef6aeda

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Uh oh!

Add webhook payload size optimization options #35129

Add webhook payload size optimization options #35129

Uh oh!

kerwin612 commented Jul 19, 2025

Uh oh!

wxiaoguang commented Jul 19, 2025

Uh oh!

kerwin612 commented Jul 23, 2025 •

edited

Loading

Uh oh!

wxiaoguang commented Jul 30, 2025 •

edited

Loading

Uh oh!

kerwin612 commented Aug 4, 2025

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

wxiaoguang Aug 28, 2025

Uh oh!

Uh oh!

wxiaoguang Aug 28, 2025

Uh oh!

wxiaoguang Aug 28, 2025 •

edited

Loading

Uh oh!

Uh oh!

Uh oh!

Add webhook payload size optimization options #35129

Are you sure you want to change the base?

Add webhook payload size optimization options #35129

Uh oh!

Conversation

kerwin612 commented Jul 19, 2025

Uh oh!

wxiaoguang commented Jul 19, 2025

Uh oh!

kerwin612 commented Jul 23, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

wxiaoguang commented Jul 30, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

kerwin612 commented Aug 4, 2025

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

wxiaoguang Aug 28, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

wxiaoguang Aug 28, 2025

Choose a reason for hiding this comment

Uh oh!

wxiaoguang Aug 28, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

kerwin612 commented Jul 23, 2025 •

edited

Loading

wxiaoguang commented Jul 30, 2025 •

edited

Loading

wxiaoguang Aug 28, 2025 •

edited

Loading