Fix wrong identify poster on a migrated pull request when submi… (#9827) (#9831)

fix dump non-exist log directory (#9818) (#9820)

Co-authored-by: Lauris BH <lauris@nix.lv>

Fix compare (#9808) (#9815)

Co-authored-by: techknowlogick <matti@mdranta.net>
Co-authored-by: techknowlogick <matti@mdranta.net>

Fix missing msteam webhook on organization (#9781) (#9795)

Fix #9662 (#9783)

Send tag create and push webhook when release created on UI (#8671) (#9702)

* 'update'

* Send push tag event when release created

* send tag create event while release created in UI

* update to go v1.13

* fix gofmt error

* update #8671 move release tag created hook to modules/notification/webhook due to #8802 refactoring

* use NotifyCreateRef and NotifyPushCommits instead of NotifyNewReleaseTag

* move tag notification to correct place

Co-authored-by: Benno <blueworrybear@gmail.com>

Hide credentials when submitting migration (#9102) (#9704)

through API.
Same fix, using form.CloneAddr instead of opts.CloneAddr.

Co-authored-by: Jordan <eatsleepgame@gmail.com>

Fix cache problem on dashboard (#9358) (#9703)

Co-authored-by: Lunny Xiao <xiaolunwen@gmail.com>

Never allow an empty password to validate (#9682) (#9684)

* Restore IsPasswordSet previous value

* Update models/user.go

Prevent redirect to Host (#9678) (#9680)

Branches not at ref commit ID should not be listed as Merged (#9614) (#9639)

Once a branch has been merged if the commit ID no longer equals that of
the pulls ref commit id don't offer to delete the branch on the pull screen
and don't list it as merged on branches.

Fix #9201

When looking at the pull page we should also get the commits from the refs/pulls/x/head

Fix #9158

[BugFix] Hide public repos owned by private orgs (#9616)

Changelog for 1.10.2 (#9585)

* Changelog for 1.10.2

* imprufe

* fix misspell

* Update CHANGELOG.md

[Backport] [API] Allow only specific Colums to be updated on Issue (#9539) (#9580)

* Fix #9189 - API Allow only specific Colums to be updated on Issue (#9539)

* dont insert "-1" in any case to issue.poster_id

* Make sure API cant override importand fields

* code format

* add Test for IssueEdit

* load milestone and return it on IssueEdit via API

* extend Test for TestAPIEditIssue

* fix TEST

* make sure Poster is loaded

* keep code format on backport as it is

[Backport] Fix bug when migrate from API (#8631) (#9563)

* Fix bug when migrate from API (#8631)

* fix bug when migrate from API

* fix test

* fix test

* improve

* fix error message

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

* remove unrelated

Co-authored-by: Lunny Xiao <xiaolunwen@gmail.com>

backport part of #9550 (#9564)

* add ErrReactionAlreadyExist
 * extend FindReactionsOptions
 * createReaction check if exit before create

fix 500 error for ghost avatar (#9537)

Fix repository issues pagination bug when there are more than one label filter (#9512) (#9528)

* Fix repository issues pagination bug when there are more than one label filter (#9512)

* fix merge

Fix deleted branch isn't removed when push the branch again (#9516) (#9524)

Fix missing repository status when migrating repository via API (#9511)

* Fix API migration wrong repository status

* Force push for ci

when branch is deleted after a pull request is merged, trigger webhook (#9510)

backport fix: fixed bug in GitTreeBySHA where pulling items from a page other than page 1 would fail because the wrong var was used to set the entries to return (#9482)

Fix NewCommitStatus (#9434) (#9435)

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

Use OriginalURL insead of CloneAddr in migration logging (#9418) (#9420)

CloneAddr will contain username and password credentials and they will
get stored in system notices about failed migrations (and logs if trace
is set). Replace with OriginalURL that doesn't have those.

Fix Slack webhook payload title generation to work with Mattermost (#9378) (#9404)

DefaultBranch needs to be prefixed by BranchPrefix (#9356) (#9359)

fix issue indexer not triggered when migrating a repository (#9333)

Fix bug that release attachment files not deleted when deleting repository (#9322) (#9329)

* Fix bug that release attachment files not deleted when deleting repository

* improve code

* add quote

* improve code

Fix migration releases (#9319) (#9326) (#9328)

* Only sync tags after all batches (#9319)

Signed-off-by: jolheiser <john.olheiser@gmail.com>
* Add SyncTags to uploader interface (#9326)

* Add sync tags to interface

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

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

File Edit: Author/Committer interchanged [BugFix] (#9300)

Changelog 1.10.1 (#9256)

* Changelog for 1.10.1

* format ...

* format

* add @techknowlogick s suggestion

Fix max length check and limit in multiple repo forms (#9148) (#9204)

* Fix input field max length for release, label and milestone forms

* Add max length for isseu and PR title

Properly fix displaying virtual session provider in admin panel (#9137) (#9203)

* Properly fix #7147

Although #7300 properly shadows the password from the virtual session
provider, the template displaying the provider config still presumed
that the config was JSON.

This PR updates the template and properly hides the Virtual Session
provider.

Fixes #7147

* update per @silverwind's suggestion

upgrade levelqueue to 0.1.0 (#9192) (#9199)

Fix panic when diff (#9187) (#9193)

* fix panic when diff

* improve code

Fix #9151 - smtp logger configuration sendTos should be an arra… (#9157)

* Fix #9151 - sendTos should be an array

* trimspace from the addresses

backport v1.10 - Always show Password field on link account sign in page (#9150)

Create PR on Current Repository by Default (#8670) (#9141)

* 'update'

* Send push tag event when release created

* send tag create event while release created in UI

* update to go v1.13

* fix gofmt error

* fix #8576 create pull request on current repository by default

fix race on indexer (#9136) (#9139)

Fix reCAPTCHA URL (#9119)

Fix reCAPTCHA issue

Backport: Hide given credentials for migrated repos. (#9098)

CloneAddr was being used as OriginalURL.
Now passing OriginalURL through from the form and saving it.

backport: update golang.org/x/crypto vendor to use acme v2 (#9056) (#9085)

Backport: Fix password checks on admin create/edit user (#9076) (#9081)

* Fix password checks on admin create/edit user

* Remove incorrect trimspace

fix: add search to reserved usernames (#9063) (#9065)

* fix: add search to reserved usernames

* Update integrations/user_test.go

Co-Authored-By: 6543 <24977596+6543@users.noreply.github.com>

Fix permission checks for close/reopen from commit (#8875) (#9033)

* Fix checks for close/reopen from commit

* Fix permission order

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

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.

Fix broken link to branch from issue list (#9003) (#9021)

* Fix broken link to branch from issue list

* Update templates/repo/issue/list.tmpl

fix wrong system notice when repository is empty (#9020)

Shadow password correctly for session config (#8984) (#9002)

Fix #8718

This PR shadows passwords in session config correctly by detecting
the VirtualProvider, unmarshalling the original config and then
shadowing config within that.

Update security (#8990)

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

Changelog for 1.10.0 (#8978)

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

Hotfix for review actions and notifications (#8965)

Backport 1.9.6 (#8969)

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

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

Backport #8901

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 Tests (#8938) (#8945)

Update all Github migration tests to use a new repo created just for
these tests that won't accidentially be modified by regular users
interacting with issues.

Fixes #8895

Backport: Enable punctuations ending mentions (#8889) (#8894)

* Enable punctuations ending mentions

* Improve tests

Fix issue with user.fullname (#8903)

Update Github migration test (#8896)

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...

Backport: Fix password complexity check on registration (#8887) (#8888)

* Fix registration password complexity

* Fix integration to use a complex password ;)

Backport: Fix require external registration password (#8885) (#8890)

* Fix require external registration password

* Fix ctx on error condition by @jolheiser

Fix edit content button on migrated issue content (#8877) (#8884)

Typo on a closing span tag caused edit button not to work properly on
the original issue content for a migrated issue.

Fixes #8876

Fix to close opened io resources as soon as not needed (#8839) (#8846)

* Fix to close opened io resources as soon as not needed

* Remove unneeded err checks

Fix new user form for non-local users (#8826) (#8828)

Fix commit expand button to not go to commit link (#8745) (#8825)

* Fix commit expand button to not go to commit link

* Fix message rendering to have correct HTML in result

* Fix check for empty commit message

* Code optimization

Fix SSH2 conditonal in key parsing code (#8806) (#8810)

Avoid out of bounds error by using strings.HasPrefix to check for
starting SSH2 text rather than assuming user input has at least 31
characters.

Add tests for bad input as well.

Fixes #8800

add missing "d" (#8801)

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

On windows set core.longpaths true (#8776) (#8786)

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

* 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) (#8756)

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

Changelog 1.10.0-rc2 (#8750)

* 1.10.0-rc2

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

Signed-off-by: jolheiser <john.olheiser@gmail.com>
* Update CHANGELOG.md

Co-Authored-By: jaqra <48099350+jaqra@users.noreply.github.com>

[Fix] milestone close timestamp (#8728) (#8730)

* BugFix: Update closed_date_unix colum on milestone table on close

* go fmt

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

Fix 500 when getting user as unauthenticated user (#8653) (#8663)

Backport #8653

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 there is a panic

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

* make call createMilestoneComment on newIssue func

* make OldMilestoneID 0 instead of -1

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

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

* fix test

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

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)

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

Fix template error on account page (#8562) (#8622)

Allow externalID to be UUID (#8551) (#8624)

Signed-off-by: Wenxuan Zhao <viz@linux.com>

Prevent .code-view from overriding font on icon fonts (#8614) (#8627)

Expose db.SetMaxOpenConns and allow non MySQL dbs to set conn pool params (#8528) (#8618)

* Expose db.SetMaxOpenConns and allow other dbs to set their connection params
* Add note about port exhaustion

Co-Authored-By: guillep2k <18600385+guillep2k@users.noreply.github.com>

fix emoji panel be removed bug in issue page, when the sub issue summit the duplicate emoji (#8609) (#8623)

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

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

Ensure that diff stats can scroll independently of the diff (#8581) (#8611)

This PR ensures that once opened the diff stats detail box can be scrolled independently of the diff on the compare page.

Fixes #5532

Details:

* make diff-detail-box the main container
* move file diff at the same level as diff-stats
* make diff-view options sticy again
* make diff-stats scroll if to mouch
* rm useless css info
* less: mv diff-stats to own class
* use new css class
* cleanup less file
* diff-counter: margin-right: 15px;
* make CI work
* make numbers colorful
* add sign (-/+) to numbers

webhook: set Content-Type for application/x-www-form-urlencoded (#8600)

This header is missing since switching http client from GiteaServer (`code.gitea.io/gitea/modules/httplib`) to Go-http-client/1.1 (`net.http`). The header [was added by default](https://github.com/go-gitea/gitea/blob/release/v1.8/modules/httplib/httplib.go#L301) by the former, but this is no longer true with `net.http`, so it needs to be done explicitly.

Closes: #7700

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

* 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 on pull requests when transfer head repository (#8564) (#8569)

* fix bug on pull requests when transfer head repository

* add migration and fix lint

* fix tests and add a cache check on LoadBaseRepo

Allow more than 255 characters for tokens in external_login_user tabl… (#8585)

* Allow more than 255 characters for tokens in external_login_user table (#8554)

Signed-off-by: Wenxuan Zhao <viz@linux.com>
* use old xorm repo

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

Ensure that GitRepo is set on Empty repositories (#8539) (#8541)

Both issues/new and settings/hooks/git expect `ctx.Repo.GitRepo` to be set.
This PR changes the context code to open the GitRepo.

Fixes #8538

Fix migrate mirror 500 bug (#8526) (#8530)

* fix migrate mirror 500 bug

* update backport

Fix password complexity regex for special characters (backport for v1.10.0) (#8524)

* Fix extra space

* Fix regular expression

* Fix error template name

* Simplify check code, fix default values, add test

* Fix router tests

* Fix fmt

* Fix setting and lint

* Move cleaning up code to test, improve comments

* Tidy up variable declaration

Changelog for 1.10.0-RC1 (#8510)

* Changelog for 1.10.0
* clean up | remove TESTING and DOCS sction | short BUILD section

Co-Authored-By: guillep2k <18600385+guillep2k@users.noreply.github.com>
Co-Authored-By: John Olheiser <42128690+jolheiser@users.noreply.github.com>
Co-Authored-By: zeripath <art27@cantab.net>

Fix errors in create org UI regarding team access permission. (#8506)

[skip ci] Updated translations via Crowdin

Add nofollow to sign in links (#8509)

[skip ci] Updated translations via Crowdin

Password Complexity Checks  (#6230)

Add password complexity checks. The default settings require a lowercase, uppercase, number and a special character within passwords.

Co-Authored-By: T-M-A <maxim.tkachenko@gmail.com>
Co-Authored-By: Lanre Adelowo <adelowomailbox@gmail.com>
Co-Authored-By: guillep2k <18600385+guillep2k@users.noreply.github.com>
Co-Authored-By: Lauris BH <lauris@nix.lv>

fix bug on FindExternalUsersByProvider (#8504)