Skip to content

Commit 08916df

Browse files
brechtvlbartvdbraak
authored andcommitted
BLENDER: Fine tune pages considered expensive
1 parent 7b34fe8 commit 08916df

File tree

1 file changed

+16
-16
lines changed

1 file changed

+16
-16
lines changed

routers/common/blockexpensive.go

Lines changed: 16 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -39,28 +39,28 @@ func isRoutePathExpensive(routePattern string) bool {
3939
}
4040

4141
expensivePaths := []string{
42-
// code related
42+
// code related, very expensive pages
4343
"/{username}/{reponame}/archive/",
4444
"/{username}/{reponame}/blame/",
45-
"/{username}/{reponame}/commit/",
46-
"/{username}/{reponame}/commits/",
4745
"/{username}/{reponame}/graph",
48-
"/{username}/{reponame}/media/",
49-
"/{username}/{reponame}/raw/",
50-
"/{username}/{reponame}/src/",
5146

52-
// issue & PR related (no trailing slash)
53-
"/{username}/{reponame}/issues",
54-
"/{username}/{reponame}/{type:issues}",
55-
"/{username}/{reponame}/pulls",
56-
"/{username}/{reponame}/{type:pulls}",
57-
58-
// wiki
59-
"/{username}/{reponame}/wiki/",
47+
// activity, trailing slash removed
48+
"/{username}/{reponame}/activity",
49+
}
6050

61-
// activity
62-
"/{username}/{reponame}/activity/",
51+
if !(strings.HasPrefix(routePattern, "/blender/") ||
52+
strings.HasPrefix(routePattern, "/studio/") ||
53+
strings.HasPrefix(routePattern, "/extensions/") ||
54+
strings.HasPrefix(routePattern, "/infrastructure/")) {
55+
// code related, less expensive not allowed in forks
56+
expensivePaths = append(expensivePaths,
57+
"/{username}/{reponame}/media/",
58+
"/{username}/{reponame}/commit/",
59+
"/{username}/{reponame}/commits/",
60+
"/{username}/{reponame}/src/",
61+
"/{username}/{reponame}/raw/")
6362
}
63+
6464
for _, path := range expensivePaths {
6565
if strings.HasPrefix(routePattern, path) {
6666
return true

0 commit comments

Comments
 (0)