fix

Author: wxiaoguang

modules/web/router_path.go

@@ -6,6 +6,7 @@ package web
 import (
 	"net/http"
 	"regexp"
+	"slices"
 	"strings"
 
 	"code.gitea.io/gitea/modules/container"
@@ -36,12 +37,22 @@ func (g *RouterPathGroup) ServeHTTP(resp http.ResponseWriter, req *http.Request)
 	g.r.chiRouter.NotFoundHandler().ServeHTTP(resp, req)
 }
 
+type RouterPathRegexp struct {
+	re          *regexp.Regexp
+	params      []routerPathParam
+	middlewares []any
+}
+
 // MatchPath matches the request method, and uses regexp to match the path.
-// The pattern uses "<...>" to define path parameters, for example: "/<name>" (different from chi router)
-// It is only designed to resolve some special cases which chi router can't handle.
+// The pattern uses "<...>" to define path parameters, for example, "/<name>" (different from chi router)
+// It is only designed to resolve some special cases that chi router can't handle.
 // For most cases, it shouldn't be used because it needs to iterate all rules to find the matched one (inefficient).
 func (g *RouterPathGroup) MatchPath(methods, pattern string, h ...any) {
-	g.matchers = append(g.matchers, newRouterPathMatcher(methods, pattern, h...))
+	g.MatchRegexp(methods, g.PatternRegexp(pattern), h...)
+}
+
+func (g *RouterPathGroup) MatchRegexp(methods string, patternRegexp *RouterPathRegexp, h ...any) {
+	g.matchers = append(g.matchers, newRouterPathMatcher(methods, patternRegexp, h...))
 }
 
 type routerPathParam struct {
@@ -96,8 +107,8 @@ func isValidMethod(name string) bool {
 	return false
 }
 
-func newRouterPathMatcher(methods, pattern string, h ...any) *routerPathMatcher {
-	middlewares, handlerFunc := wrapMiddlewareAndHandler(nil, h)
+func newRouterPathMatcher(methods string, patternRegexp *RouterPathRegexp, h ...any) *routerPathMatcher {
+	middlewares, handlerFunc := wrapMiddlewareAndHandler(patternRegexp.middlewares, h)
 	p := &routerPathMatcher{methods: make(container.Set[string]), middlewares: middlewares, handlerFunc: handlerFunc}
 	for method := range strings.SplitSeq(methods, ",") {
 		method = strings.TrimSpace(method)
@@ -106,6 +117,12 @@ func newRouterPathMatcher(methods, pattern string, h ...any) *routerPathMatcher
 		}
 		p.methods.Add(method)
 	}
+	p.re, p.params = patternRegexp.re, patternRegexp.params
+	return p
+}
+
+func patternRegexp(pattern string, h ...any) *RouterPathRegexp {
+	p := &RouterPathRegexp{middlewares: slices.Clone(h)}
 	re := []byte{'^'}
 	lastEnd := 0
 	for lastEnd < len(pattern) {
@@ -144,3 +161,7 @@ func newRouterPathMatcher(methods, pattern string, h ...any) *routerPathMatcher
 	p.re = regexp.MustCompile(reStr)
 	return p
 }
+
+func (g *RouterPathGroup) PatternRegexp(pattern string, h ...any) *RouterPathRegexp {
+	return patternRegexp(pattern, h...)
+}

modules/web/router_test.go

@@ -34,7 +34,7 @@ func TestPathProcessor(t *testing.T) {
 	testProcess := func(pattern, uri string, expectedPathParams map[string]string) {
 		chiCtx := chi.NewRouteContext()
 		chiCtx.RouteMethod = "GET"
-		p := newRouterPathMatcher("GET", pattern, http.NotFound)
+		p := newRouterPathMatcher("GET", patternRegexp(pattern), http.NotFound)
 		assert.True(t, p.matchPath(chiCtx, uri), "use pattern %s to process uri %s", pattern, uri)
 		assert.Equal(t, expectedPathParams, chiURLParamsToMap(chiCtx), "use pattern %s to process uri %s", pattern, uri)
 	}
@@ -108,7 +108,7 @@ func TestRouter(t *testing.T) {
 					m.Delete("", h())
 				})
 				m.PathGroup("/*", func(g *RouterPathGroup) {
-					g.MatchPath("GET", `/<dir:*>/<file:[a-z]{1,2}>`, stopMark("s2"), h("match-path"))
+					g.MatchRegexp("GET", g.PatternRegexp(`/<dir:*>/<file:[a-z]{1,2}>`, stopMark("s3")), stopMark("s2"), h("match-path"))
 				}, stopMark("s1"))
 			})
 		})
@@ -205,6 +205,12 @@ func TestRouter(t *testing.T) {
 			pathParams:  map[string]string{"username": "the-user", "reponame": "the-repo", "*": "d1/d2/fn", "dir": "d1/d2", "file": "fn"},
 			handlerMark: "s2",
 		})
+
+		testRoute(t, "GET /api/v1/repos/the-user/the-repo/branches/d1/d2/fn?stop=s3", resultStruct{
+			method:      "GET",
+			pathParams:  map[string]string{"username": "the-user", "reponame": "the-repo", "*": "d1/d2/fn", "dir": "d1/d2", "file": "fn"},
+			handlerMark: "s3",
+		})
 	})
 }
 

options/locale/locale_uk-UA.ini

@@ -1334,7 +1334,6 @@ editor.new_branch_name_desc=Назва нової гілки…
 editor.cancel=Відмінити
 editor.filename_cannot_be_empty=Назва файлу не може бути порожньою.
 editor.filename_is_invalid=Назва файлу недійсна: "%s".
-editor.commit_email=Електронна пошта коміту
 editor.invalid_commit_email=Адреса електронної пошти для коміту недійсна.
 editor.file_is_a_symlink=`"%s" - це символічне посилання. Символічні посилання не можна редагувати у веб-редакторі`
 editor.filename_is_a_directory=Назва файлу '%s' вже використовується як назва каталогу у цьому сховищі.
@@ -1370,7 +1369,6 @@ commits.signed_by_untrusted_user=Підписаний недовіреним к
 commits.signed_by_untrusted_user_unmatched=Підписано недовіреним користувачем, який не відповідає комітеру
 commits.gpg_key_id=Ідентифікатор GPG ключа
 commits.ssh_key_fingerprint=Відбиток ключа SSH
-commits.view_path=Переглянути в історії
 commits.view_file_diff=Переглянути зміни до цього файлу в цьому коміті
 
 commit.revert=Повернути до попереднього стану
@@ -1499,7 +1497,6 @@ issues.filter_project=Проєкт
 issues.filter_project_all=Всі проєкти
 issues.filter_project_none=Проєкт відсутній
 issues.filter_assignee=Виконавець
-issues.filter_assignee_any_assignee=Призначено будь-кому
 issues.filter_poster=Автор
 issues.filter_user_placeholder=Пошук користувачів
 issues.filter_user_no_select=Усі користувачі
@@ -1530,16 +1527,13 @@ issues.action_milestone=Етап
 issues.action_milestone_no_select=Етап відсутній
 issues.action_assignee=Виконавець
 issues.action_assignee_no_select=Немає виконавця
-issues.action_check=Встановити/зняти позначку
-issues.action_check_all=Встановити/зняти позначку з усіх елементів
 issues.opened_by=%[1]s відкрито <a href="%[2]s">%[3]s</a>
 issues.opened_by_fake=%[1]s відкрито користувачем %[2]s
 issues.previous=Попередній
 issues.next=Далі
 issues.open_title=Відкрито
 issues.closed_title=Закрито
 issues.draft_title=Чернетка
-issues.num_comments_1=%d коментар
 issues.num_comments=%d коментарів
 issues.commented_at=`прокоментував(ла) <a href="#%s">%s</a>`
 issues.delete_comment_confirm=Ви впевнені, що хочете видалити цей коментар?
@@ -1548,7 +1542,6 @@ issues.context.quote_reply=Цитувати відповідь
 issues.context.reference_issue=Посилання в новій задачі
 issues.context.edit=Редагувати
 issues.context.delete=Видалити
-issues.close=Закрити задачу
 issues.comment_manually_pull_merged_at=вручну об'єднав(-ла) коміти %[1]s в %[2]s %[3]s
 issues.close_comment_issue=Закрити з коментарем
 issues.reopen_issue=Відкрити знову
@@ -1575,7 +1568,6 @@ issues.role.collaborator=Співавтор
 issues.role.collaborator_helper=Цей користувач був запрошений до співпраці у сховищі.
 issues.role.first_time_contributor=Учасник, який вперше долучився
 issues.role.first_time_contributor_helper=Це перший внесок цього користувача в сховищі.
-issues.role.contributor=Співавтор
 issues.role.contributor_helper=Цей користувач раніше вже вносив зміни до сховища.
 issues.re_request_review=Повторно попросити рецензію
 issues.is_stale=З часу останньої перевірки в цей PR було внесено деякі зміни
@@ -2198,7 +2190,6 @@ settings.webhook.response=Відповідь
 settings.webhook.headers=Заголовки
 settings.webhook.payload=Зміст
 settings.webhook.body=Тіло
-settings.webhook.replay.description=Повторити цей веб-хук.
 settings.githook_edit_desc=Якщо хук неактивний, буде показано зразок вмісту. Якщо залишити вміст порожнім, хук буде вимкнено.
 settings.githook_name=Назва хуку
 settings.githook_content=Зміст хука
@@ -2255,7 +2246,6 @@ settings.event_pull_request_sync=Запит на злиття синхроніз
 settings.event_pull_request_sync_desc=Запит до злиття синхронізовано.
 settings.event_package=Пакет
 settings.branch_filter=Фільтр гілок
-settings.authorization_header=Заголовок авторизації
 settings.active=Активний
 settings.active_helper=Інформацію про викликані події буде надіслано за цією веб-хук URL-адресою.
 settings.add_hook_success=Веб-хук було додано.
@@ -2304,7 +2294,6 @@ settings.protect_disable_push=Заборонити Push
 settings.protect_disable_push_desc=Для цієї гілки буде заборонено виконання push.
 settings.protect_enable_push=Дозволити Push
 settings.protect_enable_push_desc=Будь-хто із правом запису зможе виконувати push для цієї гілки (за виключенням force push).
-settings.protect_enable_merge=Увімкнути об’єднання
 settings.protect_check_status_contexts=Увімкнути перевірку стану
 settings.protect_status_check_patterns=Шаблони перевірки стану:
 settings.protect_status_check_patterns_desc=Введіть шаблони, щоб вказати, які перевірки стану повинні пройти гілки, перш ніж їх буде об'єднано у гілку, що відповідає цьому правилу. Кожен рядок визначає шаблон. Шаблони не можуть бути порожніми.
@@ -2338,8 +2327,6 @@ settings.block_outdated_branch_desc=Об'єднання буде неможли
 settings.block_admin_merge_override=Адміністратори повинні дотримуватися правил захисту гілки
 settings.block_admin_merge_override_desc=Адміністратори повинні дотримуватися правил захисту гілки і не можуть їх обійти.
 settings.default_branch_desc=Обрати типову гілку сховища для запитів на злиття і комітів:
-settings.merge_style_desc=Стилі об'єднання
-settings.default_merge_style_desc=Типовий стиль об'єднання
 settings.choose_branch=Оберіть гілку…
 settings.no_protected_branch=Немає захищених гілок.
 settings.edit_protected_branch=Редагувати
@@ -2361,14 +2348,12 @@ settings.chat_id=ID чату
 settings.matrix.homeserver_url=URL домашньої сторінки
 settings.matrix.room_id=ID кімнати
 settings.matrix.message_type=Тип повідомлення
-settings.visibility.public.button=Зробити публічним
 settings.archive.header=Відправити репозиторій в архів
 settings.archive.success=Сховище успішно заархівовано.
 settings.archive.error=Сталася помилка при спробі архівувати репозиторій. Докладнішу інформацію дивіться у журналі.
 settings.archive.error_ismirror=Неможливо архівувати дзеркальне сховище.
 settings.archive.branchsettings_unavailable=Параметри гілки не доступні, якщо репозиторій архівний.
 settings.archive.tagsettings_unavailable=Параметри міток недоступні, якщо репозиторій архівний.
-settings.unarchive.header=Розархівувати це сховище
 settings.unarchive.success=Сховище успішно розархівовано.
 settings.update_avatar_success=Аватар репозиторію оновлений.
 settings.lfs=LFS
@@ -2452,7 +2437,6 @@ diff.protected=Захищений
 diff.image.side_by_side=Поруч
 diff.image.swipe=Провести пальцем
 diff.image.overlay=Накласти
-diff.has_escaped=Цей рядок містить приховані символи Юнікоду
 diff.show_file_tree=Показати дерево файлів
 diff.hide_file_tree=Сховати дерево файлів
 diff.submodule_added=Підмодуль %[1]s додано в %[2]s
@@ -2765,7 +2749,6 @@ dashboard.resync_all_hooks=Заново синхронізувати хуки п
 dashboard.reinit_missing_repos=Заново ініціалізувати всі відсутні сховища Git'а, для яких існують записи
 dashboard.sync_external_users=Синхронізувати дані зовнішніх користувачів
 dashboard.cleanup_hook_task_table=Очистити таблицю hook_task
-dashboard.cleanup_actions=Очищення ресурсів прострочених дій
 dashboard.server_uptime=Час роботи сервера
 dashboard.current_goroutine=Поточна кількість Goroutines
 dashboard.current_memory_usage=Поточне використання пам'яті
@@ -2795,11 +2778,7 @@ dashboard.total_gc_time=Загальна пауза збирача сміття
 dashboard.total_gc_pause=Загальна пауза збирача сміття (GC)
 dashboard.last_gc_pause=Остання пауза збирача сміття (GC)
 dashboard.gc_times=Кількість запусків збирача сміття (GC)
-dashboard.delete_old_actions=Видалити всі старі дії з бази даних
 dashboard.update_checker=Перевірка оновлень
-dashboard.gc_lfs=Збір сміття мета-об'єктів LFS
-dashboard.cancel_abandoned_jobs=Скасувати покинуті завдання
-dashboard.start_schedule_tasks=Запуск запланованих завдань
 dashboard.sync_branch.started=Розпочато синхронізацію гілок
 dashboard.rebuild_issue_indexer=Перебудувати індексатор задач
 dashboard.sync_repo_licenses=Синхронізувати ліцензії сховища
@@ -2872,7 +2851,6 @@ emails.not_updated=Не вдалось оновити адресу електр
 emails.duplicate_active=Ця адреса електронної пошти вже активна для іншого користувача.
 emails.change_email_header=Редагувати властивості електронної пошти
 emails.change_email_text=Ви впевнені, що хочете оновити адресу електронної пошти?
-emails.delete=Видалити адресу електронної пошти
 emails.delete_desc=Ви впевнені, що хочете видалити адресу електронної пошти?
 emails.deletion_success=Адресу електронної пошти видалено.
 emails.delete_primary_email_error=Ви не можете видалити основну адресу електронної пошти.
@@ -2893,7 +2871,6 @@ repos.issues=Задачі
 repos.size=Розмір
 repos.lfs_size=Розмір LFS
 
-packages.package_manage_panel=Керування пакетами
 packages.total_size=Загальний розмір: %s
 packages.unreferenced_size=Розмір без посилань: %s
 packages.cleanup=Очистити прострочені дані
@@ -3416,7 +3393,6 @@ settings.link=Прив'язати пакет до сховища
 settings.link.description=Якщо ви зв'яжете пакет зі сховищем, його буде вказано у списку пакетів сховища.
 settings.link.select=Обрати сховище
 settings.link.button=Оновити посилання на сховище
-settings.link.error=Не вдалося оновити посилання на сховище.
 settings.delete=Видалити пакет
 settings.delete.description=Видалення пакета є остаточним і не може бути скасоване.
 settings.delete.notice=Ви збираєтесь видалити %s (%s). Цю операцію неможливо скасувати, ви впевнені?
@@ -3540,7 +3516,6 @@ logs.always_expand_running=Завжди розгортати поточні жу
 [projects]
 deleted.display_name=Видалений проєкт
 type-1.display_name=Індивідуальний проєкт
-type-2.display_name=Проєкт сховища
 type-3.display_name=Проєкт організації
 enter_fullscreen=Повноекранний режим
 exit_fullscreen=Вийти з повноекранного режиму