Prioritize "readme.md" (#5691)

* prioritize readme.md

* Improve IsReadmeFile

* Add more tests

Ensure that sessions are passed into queries that could use the database to prevent deadlocks (#5718)

* Fixed deadlock in CreateComment

* Fix possible deadlock in UpdateIssueDeadline from createDeadlineComment

* Ensure that calls to IsTimeTracker enabled are called within session

Signed-off-by: Andrew Thornton <art27@cantab.net>
* Ensure that calls to reactionList are also called within session

Signed-off-by: Andrew Thornton <art27@cantab.net>
* Ensure all calls in NewPullRequest with the session are called within the session

Signed-off-by: Andrew Thornton <art27@cantab.net>
* Deal with potential deadlocks in repo

Signed-off-by: Andrew Thornton <art27@cantab.net>
* Ensure that isStaring is checked within our transaction

Signed-off-by: Andrew Thornton <art27@cantab.net>
* Fix mistake in isOrganizationMember

Sorry.

Add flag to skip repository dumping (#5695)

Discord Oauth2 support (#4476)

* add discord auth

* add vendor for discord

* fix syntax error

* make fmt

* update version of goth in use

* update markbates/goth

Refactor mail notification (#5110)

* mail notification implement interface

* fix file comment year

* use NullNotifier as parent struct of notifiers

fix: use correct value for "MSpan Structures Obtained" #4742 (#5706)

Signed-off-by: Julian Tölle <julian.toelle97@gmail.com>

Fix migration for user defined themes (#5682)

Do not display the raw OpenID error in the UI (#5705)

* Do not display the raw OpenID error in the UI

If there are no `WHITELIST_URIS` or `BLACKLIST_URIS` set in the openid
section of the app.ini, it is possible that gitea can leak sensitive
information about the local network through the error provided by the
UI. This PR hides the error information and logs it.

Fix #4973

Signed-off-by: Andrew Thornton <art27@cantab.net>
* Update auth_openid.go

Place error log within the `err != nil` branch.

[skip ci] Updated translations via Crowdin

docker: stop modifying file permission before migrating database (#5707)

allow user fork repo even when as a guest.. This will prompt the user to login and if authentication was successful, the user will be redirected to fork the repo (#5690)

[skip ci] Updated translations via Crowdin

Add docs about database migration (#5661)

* Add docs about database migration

* Reorder upgrade from Gogs steps

Update xorm to fix issue #5659 and #5651 (#5680)

When redirecting clean the path to avoid redirecting to //www.othersite.com (#5669)

Fix #5627

Signed-off-by: Andrew Thornton <art27@cantab.net>

fix public will not be reused as public key after deleting as deploy key (#5671)

document command line flags (#5675)

[skip ci] Updated translations via Crowdin

Allow for user specific themes (#5668)

* add migration and basic UI for changing a user's theme

* update user themem

* use right text on button

* load theme based on users' selection

* load theme based on users' selection in pwa too

* update sample config

* delete older theme loading

* implement AfterLoad to set users' theme properly

* set up default theme when creating a user. This uses the installation wide theme

* use flash messages for error

* set default theme when creating a user from the cli

* fix @lunny review

Translate home template to Czech (#5665)

Signed-off-by: Jan Breuer <jan.breuer@jaybee.cz>

Added URL mapping for Release attachments like on github.com (#1707)

Display branch name in delete branch confirmation modal. (#5654)

Fixes: #5630.

Only count users own actions for heatmap contributions (#5647)

Signed-off-by: Julian Tölle <julian.toelle97@gmail.com>

fix commit page showing status for current default branch (#5650)

Signed-off-by: Julian Tölle <julian.toelle97@gmail.com>

migrate database if app.ini found (#5290)

* migrate database if app.ini found

* replacing hard-coded user id by env variable

* Update per @zeripath's feedback

Add docs on passing SSH through to container (#5019)

* Add docs on passing SSH through to container

Wasn't clear how to pass SSH connections through the container, found a
few blog posts which described roughly the same procedure I've
documented here.

Credit should really go to:

* https://blog.sakuragawa.moe/gitea-in-docker-container-and-sharing-ssh-with-host/
* http://www.ateijelo.com/blog/2016/07/09/share-port-22-between-docker-gogs-ssh-and-local-system

Signed-off-by: Dane Elwell <dane.elwell@ukfast.co.uk>
* Add note to resolve @lafriks feedback

Fix sqlite deadlock when assigning to a PR (#5640)

* Fix sqlite deadlock when assigning to a PR

Fix 5639

Signed-off-by: Andrew Thornton <art27@cantab.net>
* More possible deadlocks found and fixed

Signed-off-by: Andrew Thornton <art27@cantab.net>

Add changelog for 1.6.3 and 1.7.0-rc2 (#5636)

* Add changelog for 1.6.3 and 1.7.0-rc2

Signed-off-by: Jonas Franz <info@jonasfranz.software>
* Add missing point for 1.6.3

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

[skip ci] Updated translations via Crowdin

SECURITY: protect DeleteFilePost et al with cleanUploadFileName (#5631)

This commit wraps more of the TreePaths with cleanUploadFileName

Signed-off-by: Andrew Thornton <art27@cantab.net>

[skip ci] Updated translations via Crowdin

Don't close issues via commits on non-default branch. (#5622)

Adds a small check to close the issues only if the referencing commits
are on the default branch.

Fixes: #2314.

Documentation: Clarity for HTTPS setups (#5626)

[https-setup]
- Made it clearer that HTTP redirection is possible
[config-cheat-sheet]
- Clarified the behavihour of the redirection-related config keys

[skip ci] Updated translations via Crowdin

Add changelog for 1.7.0-rc1 (#5616)

* Add changelog for 1.7.0-rc1

* Change position of refactoring of heatmap

* Refactoring some items in changelog

* Fix wrong PR title in changelog

Signed-off-by: Jonas Franz <info@jonasfranz.software>
* Remove backported PRs

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

Signed-off-by: Jonas Franz <info@jonasfranz.software>
* Add new PR to list

* Security fixes have been backported

Fix bug on modifying sshd username (#5624)

Should fix #5623

Update @jonasfranz's username (#5619)

* Update @jonasfranz's username

* lowercase'd username

Update owners & Date in contributing (#5620)

[skip ci] Updated translations via Crowdin

branch: Trigger update when deleting branch via UI. (#5617)

Fixes: #5309.

Issue is not overdue when it is on the same date #5566 (#5568)

* Due date time of issues and milestones is set to 23:59:59

* Add docs

* make gen swagger

* fix swagger gen

mirror: Delete tags in mirror which are removed for original repo. (#5609)

This bug was being caused by an error in the logic in `release.go`.
Credit to @yasuokav for tracing the root of the issue.

Fixes: #5192.

update v71.go to resolve #5595 (#5613)

[skip ci] Updated translations via Crowdin

Fix wrong text getting saved on editing second comment on an issue. (#5608)

* comments: Fix an incorrent DOM element selection.

This commit fixes a bug that was causing text from previously edited
comment to get saved when two comments were edited one after other.
Text area with id of `#content` isn't unique on the page but it was
being treated as unique by the event handling code.

Fixes: #5581.
* templates: Remove `id` from textarea in commit edit form.

An element is assigned an `id` only if it is unique for the whole page
but in this case there can be multiple textarea so it should have one.

Webhook for Pull Request approval/rejection (#5027)

Synchronize SSH keys on login with LDAP + Fix SQLite deadlock on ldap ssh key deletion (#5557)

* Synchronize SSH keys on login with LDAP

* BUG: Fix hang on sqlite during LDAP key deletion

LDAP via simple auth separate bind user and search base (#5055)

Delete organization endpoint added (#5601)

* Delete organization endpoint added

* Parameters added in comment

* Typo fix

* Newline character removed

fix nil pointer when adding a due date  (#5587)

* fix nil pointer

* remove nil check and just call loadRepo regardless

Make sure argsSet verifies string isn't empty too (#4980)

Add rebase with merge commit merge style (#3844) (#4052)

Signed-off-by: Julian Tölle <julian.toelle97@gmail.com>

Remove MTecknology from MAINTAINERS (#5599)

[skip ci] Updated translations via Crowdin

Fix route in swagger (#5598)

[skip ci] Updated translations via Crowdin

Gitlab does have discord integration now (#5589)

drone/mssql: use golang 1.11 build like the other tests (#5586)

Fix translation errors in doc advanced part (zh-cn) (#5112)

* ZH-CN translation of Advanced part

* Fix translation errors

* Resolve issues

backport 1.6.2 changelog (#5576)

Fix type mismatch of format string (#5574)

Immediate fix to htmlEncode user added text (#5570)

There are likely problems remaining with the way that initCommentForm
is creating its elements. I suspect that a malformed avatar url could
be used maliciously.

fix bug on upload file name (#5571)

Create stale bot config (#5243)

[skip ci] Updated translations via Crowdin

fix table name typo on SQL (#5562)

* fix table name typo on SQL

* fix reserved word user when on mssql

fix indexer reindex bug when gitea restart (#5563)

* fix issue indexer bug reindex when restart gitea

* also fix code indexer reindex when gitea restart

Update Licenses (#5558)

fix lfs version check warning log when using ssh protocol (#5501)

Support reverse proxy providing email (#5554)

This PR implements #2347

fix forgot removed records when deleting user (#5429)

* fix forgot removed records when deleting user

* fix migration

* fix rewritekey lock on sqlite

* remove unused codes

Add base repo nil check (#5555)

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

[skip ci] Updated translations via Crowdin

Fix makefile generate buildstep (#5556)

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

Improve performance of dashboard (#4977)

[skip ci] Updated translations via Crowdin

Remove a double slash in the HTTPS redirection when Let's Encrypt is enabled (#5537)

Before:

$ curl 0.0.0.0:3001
<a href="https://gitea.example.com:3000//">Found</a>.

After:

$ curl 0.0.0.0:3001
<a href="https://gitea.example.com:3000/">Found</a>.

Fixes #5536

[skip ci] Updated translations via Crowdin

fix bug when a read perm user to edit his issue (#5516)

fix detect force push failure on deletion of protected branches (#5522)

Added test environment for mssql (#4282)

* Added test environment for m$sql

* Added template for test environment for m$sql

* Fix password

* Fix password (again)

* Fix password (again again)

* Fix db

* Ci trigger (Looking at you drone....)

* Ci trigger (Looking at you drone....)

* Ci trigger (Looking at you drone....)

* Ci trigger (Looking at you drone....)

* Create master database for mssql integration tests

Signed-off-by: Jonas Franz <info@jonasfranz.software>
* Create database only if master do not exist

Signed-off-by: Jonas Franz <info@jonasfranz.software>
* Fix mssql integration tests by using custom database "gitea"

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

* bump xorm

* updated xorm

* Fixed build

fix approvals limitation (#5521)

fix permission check on api create org (#5523)

fix clone wiki failed via ssh (#5503)

fix adding reaction fail for read permission (#5515)

add git protocol v2 support via SSH on Docker image (#5520)

* add git protocol v2 support via SSH on Docker image

* remove new layer on dockerfile

Fix the Let's Encrypt handler by listening on a valid address (#5525)

* Fix the Let's Encrypt handler by listening on a valid address

Also handle errors in the HTTP server go routine, return a fatal error
when something goes wrong.

Thanks to @gbl08ma for finding the actual bug

Here is an example of the error handling:

    2018/12/11 14:23:07 [....io/gitea/cmd/web.go:87 func1()] [E] Failed to
    start the Let's Encrypt handler on port 30: listen tcp 0.0.0.0:30: bind:
    permission denied

Closes #5280

* Fix a typo

Approvals at Branch Protection (#5350)

* Add branch protection for approvals

Signed-off-by: Jonas Franz <info@jonasfranz.software>
* Add required approvals

Signed-off-by: Jonas Franz <info@jonasfranz.software>
* Add missing comments and fmt

Signed-off-by: Jonas Franz <info@jonasfranz.software>
* Add type = approval and group by reviewer_id to review

* Prevent users from adding negative review limits

* Add migration for approval whitelists

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

Fixing MSSQL timestamp type (#5511)

MSSQL is using the wrong type here which results in a strconv.ParseInt: parsing "2018-12-07T00:00:00Z": invalid syntax error.
The added datediff(SECOND, '19700101', x) results in the unix timestamp to be returned.

Signed-off-by: Christopher Dziomba <christopher.dziomba@gmail.com>

[skip ci] Updated translations via Crowdin

fix code review on mssql (#5502)

[skip ci] Updated translations via Crowdin

fix forgot deletion of notification when delete repository (#5506)

Fix empty wiki (#5504)

* fix wiki page when wiki path is exist but empty

* improve the error check

backport 1.6.1 changelog (#5507)

Improve team members and repositories settings UI (#5457)

* improve team members and repositories settings UI

* use tab on team pages

* add default description on team members and repos

* add blank on numbers and texts

* improve translation

add tests for api user orgs (#5494)

* add tests for api user orgs

* add permission for admin to list user's orgs even he is a private user of org

fix topic name length on database (#5493)

Allow link verification for services like Mastodon (#5481)

[skip ci] Updated translations via Crowdin