Merge remote-tracking branch 'giteaofficial/main'

* giteaofficial/main:
  Don't clean up hardcoded `tmp` (go-gitea#18983)
  git backend ignore replace objects (go-gitea#18979)
  Improve the deletion of issue (go-gitea#18945)
  Add note to GPG key response if user has no keys (go-gitea#18961)
  adds restore docs for docker based instances (go-gitea#18844)
  Refactor admin user filter query parameters (go-gitea#18965)

custom/conf/app.example.ini

@@ -890,7 +890,7 @@ PATH =
 ;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;
 ;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;
 ;;
-;; Path for local repository copy. Defaults to `tmp/local-repo`
+;; Path for local repository copy. Defaults to `tmp/local-repo` (content gets deleted on gitea restart)
 ;LOCAL_COPY_PATH = tmp/local-repo
 
 ;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;
@@ -902,7 +902,7 @@ PATH =
 ;; Whether repository file uploads are enabled. Defaults to `true`
 ;ENABLED = true
 ;;
-;; Path for uploads. Defaults to `data/tmp/uploads` (tmp gets deleted on gitea restart)
+;; Path for uploads. Defaults to `data/tmp/uploads` (content gets deleted on gitea restart)
 ;TEMP_PATH = data/tmp/uploads
 ;;
 ;; Comma-separated list of allowed file extensions (`.zip`), mime types (`text/plain`) or wildcard type (`image/*`, `audio/*`, `video/*`). Empty value or `*/*` allows all types.

docs/content/doc/advanced/config-cheat-sheet.en-us.md

@@ -107,7 +107,7 @@ Values containing `#` or `;` must be quoted using `` ` `` or `"""`.
 ### Repository - Upload (`repository.upload`)
 
 - `ENABLED`: **true**: Whether repository file uploads are enabled
-- `TEMP_PATH`: **data/tmp/uploads**: Path for uploads (tmp gets deleted on Gitea restart)
+- `TEMP_PATH`: **data/tmp/uploads**: Path for uploads (content gets deleted on Gitea restart)
 - `ALLOWED_TYPES`: **\<empty\>**: Comma-separated list of allowed file extensions (`.zip`), mime types (`text/plain`) or wildcard type (`image/*`, `audio/*`, `video/*`). Empty value or `*/*` allows all types.
 - `FILE_MAX_SIZE`: **3**: Max size of each file in megabytes.
 - `MAX_FILES`: **5**: Max number of files per upload
@@ -144,7 +144,7 @@ Values containing `#` or `;` must be quoted using `` ` `` or `"""`.
 
 ## Repository - Local (`repository.local`)
 
-- `LOCAL_COPY_PATH`: **tmp/local-repo**: Path for temporary local repository copies. Defaults to `tmp/local-repo`
+- `LOCAL_COPY_PATH`: **tmp/local-repo**: Path for temporary local repository copies. Defaults to `tmp/local-repo` (content gets deleted on Gitea restart)
 
 ## Repository - MIME type mapping (`repository.mimetype_mapping`)
 

docs/content/doc/usage/backup-and-restore.en-us.md

@@ -95,3 +95,59 @@ Repository Git Hooks should be regenerated if installation method is changed (eg
 With Gitea running, and from the directory Gitea's binary is located, execute: `./gitea admin regenerate hooks`
 
 This ensures that application and configuration file paths in repository Git Hooks are consistent and applicable to the current installation. If these paths are not updated, repository `push` actions will fail.
+
+### Using Docker (`restore`)
+
+There is also no support for a recovery command in a Docker-based gitea instance. The restore process contains the same steps as described in the previous section but with different paths.
+
+Example:
+
+```sh
+# open bash session in container
+docker exec --user git -it 2a83b293548e bash
+# unzip your backup file within the container
+unzip gitea-dump-1610949662.zip
+cd gitea-dump-1610949662
+# restore the gitea data
+mv data/* /data/gitea
+# restore the repositories itself
+mv repos/* /data/git/repositories/
+# adjust file permissions
+chown -R git:git /data
+# Regenerate Git Hooks
+/usr/local/bin/gitea -c '/data/gitea/conf/app.ini' admin regenerate hooks
+```
+
+The default user in the gitea container is `git` (1000:1000). Please replace `2a83b293548e` with your gitea container id or name.
+
+These are the default paths used in the container:
+
+```text
+DEFAULT CONFIGURATION:
+ CustomPath: /data/gitea (GITEA_CUSTOM)
+ CustomConf: /data/gitea/conf/app.ini
+ AppPath: /usr/local/bin/gitea
+ AppWorkPath: /usr/local/bin
+```
+
+### Using Docker-rootless (`restore`)
+
+The restore workflow in Docker-rootless containers differs only in the directories to be used:
+
+```sh
+# open bash session in container
+docker exec --user git -it 2a83b293548e bash
+# unzip your backup file within the container
+unzip gitea-dump-1610949662.zip
+cd gitea-dump-1610949662
+# restore the app.ini
+mv data/conf/app.ini /etc/gitea/app.ini
+# restore the gitea data
+mv data/* /var/lib/gitea
+# restore the repositories itself
+mv repos/* /var/lib/gitea/git/repositories
+# adjust file permissions
+chown -R git:git /etc/gitea/app.ini /var/lib/gitea
+# Regenerate Git Hooks
+/usr/local/bin/gitea -c '/etc/gitea/app.ini' admin regenerate hooks
+```

integrations/user_test.go

@@ -121,6 +121,7 @@ func TestExportUserGPGKeys(t *testing.T) {
  defer prepareTestEnv(t)()
  // Export empty key list
  testExportUserGPGKeys(t, "user1", `-----BEGIN PGP PUBLIC KEY BLOCK-----
+Note: This user hasn't uploaded any GPG keys.
 
 
 =twTO

models/issue.go

@@ -2043,17 +2043,17 @@ func deleteIssue(ctx context.Context, issue *Issue) error {
  }
 
  // delete actions assigned to this issue
- var comments []int64
- if err := e.Table(new(Comment)).In("issue_id", issue.ID).Cols("id").Find(&comments); err != nil {
+ subQuery := builder.Select("`id`").
+ From("`comment`").
+ Where(builder.Eq{"`issue_id`": issue.ID})
+ if _, err := e.In("comment_id", subQuery).Delete(&Action{}); err != nil {
  return err
  }
- for i := range comments {
- if _, err := e.Where("comment_id = ?", comments[i]).Delete(&Action{}); err != nil {
- return err
- }
- }
- if _, err := e.Table("action").Where("repo_id = ?", issue.RepoID).In("op_type", ActionCreateIssue, ActionCreatePullRequest).
- Where("content LIKE ?", strconv.FormatInt(issue.ID, 10)+"|%").Delete(&Action{}); err != nil {
+
+ if _, err := e.Table("action").Where("repo_id = ?", issue.RepoID).
+ In("op_type", ActionCreateIssue, ActionCreatePullRequest).
+ Where("content LIKE ?", strconv.FormatInt(issue.ID, 10)+"|%").
+ Delete(&Action{}); err != nil {
  return err
  }
 

models/repo.go

@@ -10,7 +10,6 @@ import (
  "fmt"
  "os"
  "path"
- "path/filepath"
  "sort"
  "strconv"
  "strings"
@@ -123,7 +122,8 @@ func NewRepoContext() {
  loadRepoConfig()
  unit.LoadUnitConfig()
 
- admin_model.RemoveAllWithNotice(db.DefaultContext, "Clean up repository temporary data", filepath.Join(setting.AppDataPath, "tmp"))
+ admin_model.RemoveAllWithNotice(db.DefaultContext, "Clean up temporary repository uploads", setting.Repository.Upload.TempPath)
+ admin_model.RemoveAllWithNotice(db.DefaultContext, "Clean up temporary repositories", LocalCopyPath())
 }
 
 // CheckRepoUnitUser check whether user could visit the unit of this repository

models/user/search.go

@@ -20,6 +20,7 @@ import (
 // SearchUserOptions contains the options for searching
 type SearchUserOptions struct {
  db.ListOptions
+
  Keyword string
  Type UserType
  UID int64
@@ -33,6 +34,8 @@ type SearchUserOptions struct {
  IsRestricted util.OptionalBool
  IsTwoFactorEnabled util.OptionalBool
  IsProhibitLogin util.OptionalBool
+
+ ExtraParamStrings map[string]string
 }
 
 func (opts *SearchUserOptions) toSearchQueryBase() *xorm.Session {

modules/context/pagination.go

@@ -55,12 +55,3 @@ func (p *Pagination) SetDefaultParams(ctx *Context) {
  p.AddParam(ctx, "tab", "TabName")
  p.AddParam(ctx, "t", "queryType")
 }
-
-// SetUserFilterParams sets common pagination params for user filtering, e.g. the admin userlist
-func (p *Pagination) SetUserFilterParams(ctx *Context) {
- p.AddParamString("status_filter[is_active]", ctx.FormString("status_filter[is_active]"))
- p.AddParamString("status_filter[is_admin]", ctx.FormString("status_filter[is_admin]"))
- p.AddParamString("status_filter[is_restricted]", ctx.FormString("status_filter[is_restricted]"))
- p.AddParamString("status_filter[is_2fa_enabled]", ctx.FormString("status_filter[is_2fa_enabled]"))
- p.AddParamString("status_filter[is_prohibit_login]", ctx.FormString("status_filter[is_prohibit_login]"))
-}

modules/git/command.go

@@ -158,6 +158,8 @@ func (c *Command) RunWithContext(rc *RunContext) error {
  fmt.Sprintf("LC_ALL=%s", DefaultLocale),
  // avoid prompting for credentials interactively, supported since git v2.3
  "GIT_TERMINAL_PROMPT=0",
+ // ignore replace references (https://git-scm.com/docs/git-replace)
+ "GIT_NO_REPLACE_OBJECTS=1",
  )
 
  cmd.Dir = rc.Dir

routers/web/admin/users.go

@@ -41,10 +41,16 @@ func Users(ctx *context.Context) {
  ctx.Data["PageIsAdmin"] = true
  ctx.Data["PageIsAdminUsers"] = true
 
+ extraParamStrings := map[string]string{}
  statusFilterKeys := []string{"is_active", "is_admin", "is_restricted", "is_2fa_enabled", "is_prohibit_login"}
  statusFilterMap := map[string]string{}
  for _, filterKey := range statusFilterKeys {
- statusFilterMap[filterKey] = ctx.FormString("status_filter[" + filterKey + "]")
+ paramKey := "status_filter[" + filterKey + "]"
+ paramVal := ctx.FormString(paramKey)
+ statusFilterMap[filterKey] = paramVal
+ if paramVal != "" {
+ extraParamStrings[paramKey] = paramVal
+ }
  }
 
  sortType := ctx.FormString("sort")
@@ -68,6 +74,7 @@ func Users(ctx *context.Context) {
  IsRestricted: util.OptionalBoolParse(statusFilterMap["is_restricted"]),
  IsTwoFactorEnabled: util.OptionalBoolParse(statusFilterMap["is_2fa_enabled"]),
  IsProhibitLogin: util.OptionalBoolParse(statusFilterMap["is_prohibit_login"]),
+ ExtraParamStrings: extraParamStrings,
  }, tplUsers)
 }
 

routers/web/explore/user.go

@@ -82,7 +82,9 @@ func RenderUserSearch(ctx *context.Context, opts *user_model.SearchUserOptions,
 
  pager := context.NewPagination(int(count), opts.PageSize, opts.Page, 5)
  pager.SetDefaultParams(ctx)
- pager.SetUserFilterParams(ctx)
+ for paramKey, paramVal := range opts.ExtraParamStrings {
+ pager.AddParamString(paramKey, paramVal)
+ }
  ctx.Data["Page"] = pager
 
  ctx.HTML(http.StatusOK, tplName)

routers/web/user/home.go

@@ -736,6 +736,7 @@ func ShowGPGKeys(ctx *context.Context, uid int64) {
  ctx.ServerError("ListGPGKeys", err)
  return
  }
+
  entities := make([]*openpgp.Entity, 0)
  failedEntitiesID := make([]string, 0)
  for _, k := range keys {
@@ -755,6 +756,8 @@ func ShowGPGKeys(ctx *context.Context, uid int64) {
  headers := make(map[string]string)
  if len(failedEntitiesID) > 0 { // If some key need re-import to be exported
  headers["Note"] = fmt.Sprintf("The keys with the following IDs couldn't be exported and need to be reuploaded %s", strings.Join(failedEntitiesID, ", "))
+ } else if len(entities) == 0 {
+ headers["Note"] = "This user hasn't uploaded any GPG keys."
  }
  writer, _ := armor.Encode(&buf, "PGP PUBLIC KEY BLOCK", headers)
  for _, e := range entities {