Ensure Written is set in GZIP ProxyResponseWriter (#9018) (#9026)

Fix #9001

The GZIP ProxyReponseWriter doesn't currently respond correctly
to requests about its Written status - leading to #9001.

This PR properly reimplements these methods.

Changelog for 1.9.6 (#8967)

* Changelog for 1.9.6

Signed-off-by: jolheiser <john.olheiser@gmail.com>

Add Close() method to gogitRepository (#8901) (#8958)

Backport #8901 - Adjusted slightly for 1.9

In investigating #7947 it has become clear that the storage component of go-git repositories needs closing.

This PR adds this Close function and adds the Close functions as necessary.

In TransferOwnership the ctx.Repo.GitRepo is closed if it is open to help prevent the risk of multiple open files.

Fixes #7947

Update Github Migration Test (#8946)

Small fix for test on 1.9 since #8938 can't be easily back ported to
this branch.

Fix issue with user.fullname (#8904)

Update Github migration test (#8897)

Earlier today #716 was reopened which updated the modification time for
an old milestone (1.6.0) that we use in testing with the assumption that
it is old and won't change. This breaks all builds now, so remove this
test since we have others that test the same code and this milestone
will likely be updated again as that issue changes etc...

fix 500 when edit hook (#8782) (#8790)

Allow to merge if file path contains " or \ (#8629) (#8772)

* if a filename in a repository contains " or \ the owner can't merge pull request with this files
because "git diff-tree" adds double quotes to that filepath
example: filepath is ab"cd but "git diff-tree" returns "ab\"cd"

now, when the owner click "Merge Pull Request" button the server returns 500
this commit fix it

Signed-off-by: Ilya Pavlov <ilux@cpan.org>
* add -z option to getDiffTree
escape spec symbols for sparse-checkout

Signed-off-by: Ilya Pavlov <ilux@cpan.org>
* go fmt

Signed-off-by: Ilya Pavlov <ilux@cpan.org>
* typo

Signed-off-by: Ilya Pavlov <ilux@cpan.org>
* escape '\'
escape all spaces and '!'

* use regexp.ReplaceAllString()

Signed-off-by: Ilya Pavlov <ilux@cpan.org>
* strings.ReplaceAll was added in go 1.12

Signed-off-by: Ilya Pavlov <ilux@cpan.org>
* add '\' to regexp.MustCompile

Signed-off-by: Ilya Pavlov <ilux@cpan.org>

Changelog 1.9.5 (#8753)

* 1.9.5

Signed-off-by: jolheiser <john.olheiser@gmail.com>

[Backport] [Fix] milestone close timestamp (#8728) (#8731)

* [Fix] milestone close timestamp (#8728)

* BugFix: Update closed_date_unix colum on milestone table on close

* use go standart time lib

* make backport work!

Fix deadline on update issue or PR via API (#8699)

make call createMilestoneComment on newIssue func (#8678) (#8682)

* make call createMilestoneComment on newIssue func

* make OldMilestoneID 0 instead of -1

Revert "API should follow RequireSignInView (#8654) (#8661)" (#8674)

This reverts commit ffff835b7338a25be96df8747a336f07afc49db2.

Fix 500 when getting user as unauthenticated user (#8662)

When doing GET /api/v1/users/{user} as an unauthenticated user,
gitea throws a 500 because it's trying to dereference elements
from the context user. It wants to do this to see whether to
show the primary email and will do that if the logged in user
is admin or the user in question. However, if ctx.User is nil,
go gets really unhappy.

API should follow RequireSignInView (#8654) (#8661)

Hide some user information via API if user have no enough permission (#8655) (#8658)

* Hide some user information via API if user have no enough permission

* fix test

Use AppSubUrl for more redirections (#8647) (#8652)

Partial backport without changes to locale files.

Fix #8461 - fix misspelling of {{AppSubUrl}} and other misspelling in template
Fixes /explore and organisation redirection

Add SubURL to redirect path (#8632) (#8634) (#8640)

Signed-off-by: jolheiser <john.olheiser@gmail.com>

Update heatmap fixtures to restore tests (#8615) (#8617)

* Update heatmap fixtures to restore tests
* Add hint to check the fixture age on fail

Fix #8582 by handling empty repos (#8587) (#8593)

* Fix #8582 by handling empty repos

Signed-off-by: Jonas Franz <info@jonasfranz.software>
* Fix tests

Signed-off-by: Jonas Franz <info@jonasfranz.software>

fix bug pull request files will be broken if head repo was transfered to another user or orgnization (#8571)

Add missed close in ServeBlobLFS (#8527) (#8543)

Fixes #8369: Create .ssh dir as necessary (#8486) (#8489)

* Ensure .ssh dir exists before rewriting public keys

* Ensure .ssh dir exists before appending to authorized_keys

* Log the error because it would be useful to know where it is trying to MkdirAll

* Only try to create RootPath if it's not empty

IsBranchExist: return false if provided name is empty (#8485) (#8492)

* IsBranchExist: return false if provided name is empty

* Ensure that the reference returned is actually of a valid type

Backport: Ignore mentions for users with no access (#8395) (#8484)

* Ignore mentions for users with no access

* Fix fmt

Restore functionality for early gits (#7775) (#8476)

* Change tests to make it possible to run TestGit with 1.7.2

* Make merge run on 1.7.2

* Fix tracking and staging branch name problem

* Ensure that git 1.7.2 works on tests

* ensure that there is no chance for conflicts

* Fix-up missing merge issues

* Final rm

* Ensure LFS filters run on the tests

* Do not sign commits from temp repo

* Apply suggestions from code review

* Update modules/repofiles/temp_repo.go

Add check for empty set when dropping indexes during migration (#8475)

Ensure Request Body Readers are closed in LFS server (#8454) (#8459)

Ensure that LFS files are relative to the LFS content path (#8455) (#8458)

Changelog for v1.9.4 (#8422)

* changelog

* Update CHANGELOG.md

We ned to revert this then ...

Co-Authored-By: Lauris BH <lauris@nix.lv>

Revert "[Backport] Fix issues/pr list broken when there are man… (#8425)

This reverts commit 0ea4b786cb99054a381f3005f11f4cf413214617.

[Backport] Fix issues/pr list broken when there are many repositories (#8409) (#8418)

* Fix issues/pr list broken when there are many repositories (#8409)

* fix issues/pr list broken when there are many repositories

* remove unused codes

* fix counting error on issues/prs

* keep the old logic

* fix panic

* fix tests

* rm unused import

fix bug when migrating a private repository (#7917) (#8403)

Change general form binding to gogs form (#8334) (#8402)

feat: highlight issue references with : (#8101) (#8404)

* feat: highlight issue references with :

e.g. #1287: my commit msg
e.g. ABC-1234: my commit msg

* ref: update model regex to consistent with issueNumericPattern

* test: check highlight issue with : in commits messages

Fix milestone num_issues (#8221) (#8400)

* fix milestone num_issues

* update missing completeness

* only update milestone closed number when closed issue is assigned a new milestone or clear milestone

* fix tests

* fix update milestone num

* fix completeness calculate

* make completeness calucation more clear

Fix editor commit to new branch if PR disabled (#8375) (#8401)

Allow users with explicit read access to give approvals (#8398)

fix version-validation (because of 1.12 -> 1.13 switch) (#8389)

* fix depenedency

* make vendor first

Backport of commit status fixes in PR #8316 and PR #8321 to v1.9 (#8339)

* Use correct index when fetching commit status

Signed-off-by: Elias Norberg <elias@aisle.se>
* Compare against base repo to avoid mismatch when merging from fork

Signed-off-by: Elias Norberg <elias@aisle.se>
* Fix pull request commit status in user dashboard list

Fix API for edit and delete release attachment (#8290)

* Add logging for when user requested attachment doesn't belong to the specified release.

* Fix API to use correct variable for release asset (attachment)

Fix assets on release webhook (#8283)

Fix release API URL generation (#8239)

Allow registration when button is hidden (#8238)

MS Teams webhook misses commit messages (#8225)

The current webhook just shows the amount of commits, but misses the actual commit description. While the code is actually there to include the description, it is just not included.

Signed-off-by: Bjoern Petri <bjoern.petri@sundevil.de>

Fix data race (#8206)

* Fix data race

* Fix data race in modules/log

* Make the scope of lock finner-grained

* Use syc.Map

* Fix missing change in the test

* Do not export LoggerMap

Fix team user api (#8172) (#8188)

* fix team user api

* fix tests

* fix api

* fix team user api

* change user convert

* fix tests

* fix tests

Fix pull merge 500 error caused by git-fetch breaking behaviors (#8194)

[ssh] fix the config specification in the authorized_keys template (#8031) (#8193)

The gitea flags must appear in front of the gitea command. Otherwise
 they are ignored.

Signed-off-by: Jakob Ackermann <das7pad@outlook.com>
(cherry picked from commit 89648746a5acd4892a026d2962ff217280dbdf5b)

[backport/v1.9] Fix reading git notes from nested trees (#8189)

* Fix reading notes from nested trees

The GIT documentation for notes states "Permitted pathnames have the
form ab/cd/ef/.../abcdef...: a sequence of directory names of two
hexadecimal digits each followed by a filename with the rest of
the object ID."

* Add test case

* Fix new lines

[fix #7384] make show private icon when repo avatar set (#8144) (#8175)

Add reviewrs as participants (#8124)

Backport: Use vendored go-swagger (#8087) (#8165)

* Use vendored go-swagger (#8087)

* Use vendored go-swagger

* vendor go-swagger

* revert un wanteed change

* remove un-needed GO111MODULE

* Update Makefile

Co-Authored-By: techknowlogick <matti@mdranta.net>
* re-generate swagger file

target go1.12

Changelog for v1.9.3 (#8117)

* Changelog for 1.9.3

* Fix ref to 8071

* Correct punctuation

Co-Authored-By: techknowlogick <matti@mdranta.net>

Fix Go 1.13 private repository go get issue (#8100)

* Fix Go 1.13 invalid import path creation

Signed-off-by: Rutger Broekhoff <rutger@viasalix.nl>
* Apply suggested changes from #8100

Signed-off-by: Rutger Broekhoff <rutger@viasalix.nl>

Backport: Strict name matching for Repository.GetTagID() (#8082)

* Strict name matching for Repository.GetTagID()

* Add test for GetTagID()

Avoid ambiguity of branch/directory names for the git-diff-tree command (#8066) (#8070)

Bugfix/deformed emoji in commit message (backport 1.9) (#8071)

* Fixes deformed emoji in pull request comments or reviews

Signed-off-by: Mario Lubenka <mario.lubenka@googlemail.com>
* Generate css via command

Signed-off-by: Mario Lubenka <mario.lubenka@googlemail.com>

Add change title notification for issues (#8064)

Run CORS handler first for /api routes (#7967) (#8053)

Signed-off-by: Tamal Saha <tamal@appscode.com>

Evaluate emojis in commit messages in list view (#7906) (#8044)

Signed-off-by: Mario Lubenka <mario.lubenka@googlemail.com>
(cherry picked from commit 08c63190c68ca63f99f510f7f2c6cda41d531343)

keep blame view buttons sequence consistent with normal view when view a file (#8007) (#8009)

Backport for #7945 (#7994)

backport: Fix adding default Telegram webhook (#7992)

Signed-off-by: Gary Kim <gary@garykim.dev>

Abort syncrhonization from LDAP source if there is some error. (#7965)

Signed-off-by: David Svantesson <davidsvantesson@gmail.com>
(cherry picked from commit b2d23a138942179d62f9bffdb3df272e9300e35b)

Add release notes of 1.9.2 (#7934)

* Add release notes of 1.9.2

* add missing issue

Fix pull creation with empty changes (#7920) (#7926)

* Logs the stderr of git-apply
* Add an integration test
* Skip testPatch when patch is empty

fix wrong sender when send slack webhook (#7918) (#7924)

upload support text/plain; charset=utf8 (#7899)

drone/docker: prepare multi-arch release + provide arm64 image (#7571) (#7884)

* drone/docker: prepare multi-arch release

* Add docker-linux-arm64 pipeline

* add arm 64 build to manifest

* tag dry-run + indent

* Fix notify dependency

 lfs/lock: round locked_at timestamp to second (#7872) (#7875)

* lfs/lock: round locked_at timestamp to second

* test returned locked_at values

* tests: use time RFC3339

fix non existent milestone with 500 error (#7867) (#7873)

Add changelog of v1.9.1 (#7859)

* add changelog of v1.9.1

* Update CHANGELOG.md

Co-Authored-By: Lauris BH <lauris@nix.lv>
* mention releases build by go1.12.8

* Update CHANGELOG.md

Co-Authored-By: Lauris BH <lauris@nix.lv>

Fix local runs of ssh-requiring integration tests (#7857)

Attempt to fix hook problem (#7856)

gpg/bugfix: Use .ExpiredUnix.IsZero to display green color of forever valid gpg key (#7846) (#7850)

* Use .ExpiredUnix.IsZero for display green color of gpg key

* remove useless parentheses

Do not fetch all refs (#7837)

Which would unnecessarily slow down the pull compare operation.

Fix duplicate call of webhook (#7821) (#7824)

Enable switching to a different source branch when PR already exists (#7823)

Signed-off-by: Mario Lubenka <mario.lubenka@googlemail.com>

Rewrite existing repo units if setting is not included in api body (#7811)

Signed-off-by: David Svantesson <davidsvantesson@gmail.com>

Prevent Commit Status and Message From Overflowing On Branch Page (#7800) (#7808)

Signed-off-by: Gary Kim <gary@garykim.dev>

api: fix multiple bugs with statuses endpoints (#7785) (#7807)

* fix commit statuses api url

* search refs before passing sha

* adjust tests

* directly search tags and branches names + remove un-needed check in NewCommitStatus

* fix comment

* de-duplicate code

* test: use relative setting.AppURL

* Update routers/api/v1/repo/status.go

Co-Authored-By: Lauris BH <lauris@nix.lv>
* remove return

* Update routers/api/v1/repo/status.go

Co-Authored-By: Lauris BH <lauris@nix.lv>

Fix Slack webhook fork message (#7774) (#7783)

The order of forkee and fork was mixed up.

Fix approvals counting (#7757) (#7777)

* fix approvals counting

* fix tests

* fmt

fix rename failed when rewrite public keys (#7761) (#7769)

Fix dropTableColumns sqlite implementation (#7710) (#7765)

* Fix dropTableColumns sqlite implementation

* use droptables and its index dropping support in v78 and v85

* golang-ci fixes

* Add migration from gitea 1.3.3 for sqlite which reveals the droptables bug - thus showing this works

Be more strict with git arguments (#7715) (#7762)

* Be more strict with git arguments
* fix-up commit test
* use bindings for branch name

add pagination for admin api get orgs and fix only list public orgs bug (#7742) (#7752)

Backport: fix repo_index_status lingering when deleting a repository (#7738)

Fix milestone completness calculation when migrating (#7725) (#7732)

Fixes indexed repos keeping outdated indexes when files grow too large (#7731)

* Fixes indexed repos keeping outdated indexes when files grow too large

Co-Authored-By: zeripath <art27@cantab.net>

Backport: skip non-regular files (e.g. submodules) on repo indexing (#7717)

* Backport: skip non-regular files (e.g. submodules) on repo indexing

* Include "executable" files in the index, as they are not necessarily binary

improve branches list performance and fix protected branch icon when no-login (#7695) (#7704)

Correct wrong datetime format for git (#7689) (#7690)

1.9.0 Changelog (#7676)

fix duplicated webhook when creating issue with assignees (#7681) (#7684)

upgrade macaron/captcha to fix random error problem (#7407) (#7683)

Backport of #7675. (#7682)

Move add to hook queue for created repo to outside xorm session.

Signed-off-by: David Svantesson <davidsvantesson@gmail.com>

Show protection symbol if needed on default branch (#7660) (#7668)

* backport issue showing portection symbol if needet at default branch

* sugestion to use range .Branches
https://github.com/go-gitea/gitea/pull/7668/files/d57973a8041a741fdb7e77bc6ab18431bbf95ba4