|
| 1 | +From 782f25cfc87e87276eb8e0c701ace5d9f11bf6d7 Mon Sep 17 00:00:00 2001 |
| 2 | +From: Brecht Van Lommel < [email protected]> |
| 3 | +Date: Thu, 12 Jun 2025 14:43:47 +0200 |
| 4 | +Subject: [PATCH] BLENDER: Fine tune pages considered expensive |
| 5 | + |
| 6 | +--- |
| 7 | + routers/common/blockexpensive.go | 35 ++++++++++++++++---------------- |
| 8 | + 1 file changed, 17 insertions(+), 18 deletions(-) |
| 9 | + |
| 10 | +diff --git a/routers/common/blockexpensive.go b/routers/common/blockexpensive.go |
| 11 | +index ba11ff4cf9..32c60ef64c 100644 |
| 12 | +--- a/routers/common/blockexpensive.go |
| 13 | ++++ b/routers/common/blockexpensive.go |
| 14 | +@@ -39,29 +39,28 @@ func isRoutePathExpensive(routePattern string) bool { |
| 15 | + } |
| 16 | + |
| 17 | + expensivePaths := []string{ |
| 18 | +- // code related |
| 19 | ++ // code related, very expensive pages |
| 20 | + "/{username}/{reponame}/archive/", |
| 21 | + "/{username}/{reponame}/blame/", |
| 22 | +- "/{username}/{reponame}/commit/", |
| 23 | +- "/{username}/{reponame}/commits/", |
| 24 | + "/{username}/{reponame}/graph", |
| 25 | +- "/{username}/{reponame}/media/", |
| 26 | +- "/{username}/{reponame}/raw/", |
| 27 | +- "/{username}/{reponame}/src/", |
| 28 | + |
| 29 | +- // issue & PR related (no trailing slash) |
| 30 | +- "/{username}/{reponame}/issues", |
| 31 | +- "/{username}/{reponame}/{type:issues}", |
| 32 | +- "/{username}/{reponame}/pulls", |
| 33 | +- "/{username}/{reponame}/{type:pulls}", |
| 34 | +- "/{username}/{reponame}/{type:issues|pulls}", // for 1.23 only |
| 35 | +- |
| 36 | +- // wiki |
| 37 | +- "/{username}/{reponame}/wiki/", |
| 38 | +- |
| 39 | +- // activity |
| 40 | +- "/{username}/{reponame}/activity/", |
| 41 | ++ // activity, trailing slash removed |
| 42 | ++ "/{username}/{reponame}/activity", |
| 43 | + } |
| 44 | ++ |
| 45 | ++ if !(strings.HasPrefix(routePattern, "/blender/") || |
| 46 | ++ strings.HasPrefix(routePattern, "/studio/") || |
| 47 | ++ strings.HasPrefix(routePattern, "/extensions/") || |
| 48 | ++ strings.HasPrefix(routePattern, "/infrastructure/")) { |
| 49 | ++ // code related, less expensive not allowed in forks |
| 50 | ++ expensivePaths = append(expensivePaths, |
| 51 | ++ "/{username}/{reponame}/media/", |
| 52 | ++ "/{username}/{reponame}/commit/", |
| 53 | ++ "/{username}/{reponame}/commits/", |
| 54 | ++ "/{username}/{reponame}/src/", |
| 55 | ++ "/{username}/{reponame}/raw/") |
| 56 | ++ } |
| 57 | ++ |
| 58 | + for _, path := range expensivePaths { |
| 59 | + if strings.HasPrefix(routePattern, path) { |
| 60 | + return true |
| 61 | +-- |
| 62 | +2.45.2 |
| 63 | + |
0 commit comments