backport 5571 (#5573)

fix indexer reindex bug when gitea restart (#5563) (#5564)

* fix issue indexer bug reindex when restart gitea

* also fix code indexer reindex when gitea restart

Backport #5537 Remove a double slash in the HTTPS redirect with Let's Encrypt (#5539)

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

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

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

Backported #5525 Fix the Let's Encrypt handler (#5527)

* 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

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

fix undeleted content when deleting user (#5509)

Fix empty wiki (#5504) (#5508)

* fix wiki page when wiki path is exist but empty

* improve the error check

Add 1.6.1 changelog (#5500)

fix topic name length on database (#5493) (#5495)

ensure that the `closed_at` is set for closed (#5450)

right now the `closed_at` field for json responses is not filled during
the `APIIssue` creation for api responses.

For a closed issue you get a result like:
```json
"state":"open","comments":0,"created_at":"2018-11-29T16:39:24+01:00",
"updated_at":"2018-11-30T10:49:19+01:00","closed_at":null,
"due_date":null,"pull_request":null}
```
which has no information about the closing date. (which exists in the
db and ui)
with this PR the result changes to this:

```json
:null,"assignee":null,"assignees":null,
"state":"closed",
"comments":0,"created_at":"2018-11-29T16:43:05+01:00",
"updated_at":"2018-12-02T19:17:05+01:00",
"closed_at":"2018-12-02T19:17:05+01:00",
"due_date":null,"pull_request":null}
```

fixes: https://github.com/go-gitea/gitea/issues/5446
Signed-off-by: Roman <romaaan.git@gmail.com>

Admin should be able to delete repos even if he is not a member of the organization (#5443) (#5447)

word-break the WebHook url to prevent a ui-break (#5445)

right now, the url is displayed with an anchor tag with no classes. If
the url is really really long, the url will break out of the containing
div and (depending on the url length) the browser shows the horizontal
scrollbar.
This pr makes use of the already existing css class `dont-break-out`
which gives all the anchor the necessary properties to prevent the
break.
Another solution could be to introduce some classes like
`text text-break-word`, but that would duplicate the `dont-break-out`
class just for text elements that use the `text` class.

fixes: https://github.com/go-gitea/gitea/issues/5416
Signed-off-by: Roman <romaaan.git@gmail.com>

Fix repository deletion when there is large number of issues in it (#5426) (#5434)

Fix dependent issue searching when gitea is run in subpath (#5392) (#5400)

Backported #5383 on v1.6: API: '/orgs/:org/repos': return private repos with read access (#5310) (#3829) (#5393)

Signed-off-by: Daniel Balko <inxonic+github@gmail.com>

add changelog of v1.6.0 (#5379)

* add changelog of v1.6.0

* improve changelog

dont' send assign webhooks when creating issue (#5365) (#5369)

Migration fixes 5318 1.6 backport (#5355)

* Remove field from migration to support upgrades from older version

That will ensure the field does not get queried in the Select if it does
not exist yet:

```
[I] [SQL] SELECT "id", "repo_id", "index", "poster_id", "name", "content", "milestone_id", "priority", "assignee_id", "is_closed", "is_pull", "num_comments", "ref", "deadline_unix", "created_unix", "updated_unix
[...itea/routers/init.go:60 GlobalInit()] [E] Failed to initialize ORM engine: migrate: do migrate: pq: column "ref" does not exist
```

see #5318

* Skip remove stale watcher migration if not required

Otherwise the migration will fail if executed from a older database
version without multiple IssueWatch feature.

```
2018/11/11 23:51:14 [I] [SQL] SELECT DISTINCT "issue_watch"."user_id", "issue"."repo_id" FROM "issue_watch" INNER JOIN issue ON issue_watch.issue_id = issue.id WHERE (issue_watch.is_watching = $1) LIMIT 50 []int
[...itea/routers/init.go:60 GlobalInit()] [E] Failed to initialize ORM engine: migrate: do migrate: pq: relation "issue_watch" does not exist
```

see #5318

Fix create team, update team missing units (#5188) (#5313)

Fix file edit change preview functionality (#5300) (#5311)

fix bug when users have serval teams with different units on different repositories (#5307) (#5308)

Fix U2F if gitea is configured in subpath (#5302) (#5306)

Fix markdown image with link (#4675) (#5299)

* Fix markdown image with link

* Add gitea copyright notice

* add a test for markdown image with link

* remove svg related variables

Add changelog for 1.5.3 release (#5227) (#5298)

Remove maxlines option for file logger (#5282) (#5287)

Backport fix broken translation (#5284)

1.6.0-RC2 Changelog (#5275)

Backport #5250 on v1.6: Fix Issue 5249 and protect /api/v1/admin routes with CSRF token (#5272)

* Add CSRF checking to reqToken and place CSRF in the post for deadline creation

Fixes #5226, #5249

* /api/v1/admin/users routes should have reqToken middleware

Backported wrong api request url for instances running in subfolders (#5247) (#5261)

fix: Accept web-command cli flags if web-command is commited (#5245)

* Added flags of default cmd CmdWeb to app-wide flags
* If command *is* specified app-wide flags are ignored

Backport of #5200
Signed-off-by: Berengar W. Lehr <Berengar.Lehr@kompetenztest.de>

This commit will reduce join star, repo_topic, topic tables on repo search, so that fix extra columns problem on mssql (#5136) (#5229)

* This commit will reduce join star, repo_topic, topic tables on repo search, so that fix extra columns problem on mssql

* fix tests

fix data race on migrate repository (#5224) (#5230)

fix: Add secret to all webhook's payload where it has been missing (#5208)

* Updated dependency manager via `dep ensure -update code.gitea.io/sdk`
* Gopkg.toml was not changed as sdk version is set to "master"
* affects webhooks for: Delete, Fork, IssueComment, Release
* also contains changes from go-gitea/go-sdk#125 and hence a swagger update

Signed-off-by: Berengar W. Lehr <Berengar.Lehr@kompetenztest.de>
Resolves: #4732, #5173

fix sqlite lock (#5210) (#5223)

Update go-macaron/session to latest mast to fix RCE-bug (#5195)

Fix race on updatesize (#5190) (#5215)

* fix race on updatesize

* fix more repoPath

fix to 3819 - Backport (#5219)

fix sqlite and mssql lock (#5214) (#5218)

Fix sqlite lock (#5176) (#5179)

* fix sqlite lock

* fix sqlite lock on getUnitType

Add comment replies (#5147)

*         Add comment replies

Signed-off-by: Jonas Franz <info@jonasfranz.software>
* Use review.ID instead

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

Fix SQL quoting (#5137)

`show` is keyword in MySQL and has to be quoted to reference a column name. Use grave accents (ASCII code 96) for quoting to match rest of the source code. It's non-standard SQL, but it's supported by SQLite and MySQL.

Signed-off-by: Filip Navara <navara@emclient.com>

Fix regex to support optional end line of old section in diff hunk (#5097)

+ Named groups in reges for easier group parsing

ZH-CN translation of Usage part (#5086)

[skip ci] Updated translations via Crowdin

Add change log for 1.6.0-rc1 (#5095)

* add change log for 1.6.0-rc1

* Fix some uppercases

[skip ci] Updated translations via Crowdin

ZH-CN translation of Advanced part (#5090)

Update process to include tag vX.Y.0-dev on master branch (#5091)

CN translation of README (#5050)

fix release creation via API (#5076)

* fix release creation via API

* fix release create tests

Fix dep for actual goth version used in dep lock file (#5078)

Fix golint to new URL (#5079)

Fixed spelling typos in CONTRIBUTING.md (#5061)

Add HOWTO for releasing Gitea (#5054)

* Add HOWTO for releasing Gitea

Used instructions from: https://github.com/go-gitea/gitea/pull/2609#issuecomment-352159155

* Update CONTRIBUTING.md

Backport 1.5.2 changelog (#5053)

Fix adding review comment in split view (#5038)

Remove links from topics in edit mode (#5026)

Fix missing AppSubUrl in few more templates (#5021)

Fix missing AppSubUrl in some templates (#5020)

Hide outdated comments in file view (#5017)

* Hide outdated comments in file view

Signed-off-by: Jonas Franz <info@jonasfranz.software>
* Add unit test by adding "invalidated" comment to fixtures

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

Fix a typo in app.ini.sample (#5015)

Add missing envionment vars for DB with Docker (#5007)

Current docs demonstrate how to configure a database container but don't
explicitly specify that you should add additional environment variables
to the Gitea container to make it use the database. This just
demonstrates the changes required.

Signed-off-by: Dane Elwell <dane.elwell@ukfast.co.uk>

README: update make command to build full release (#5004)

* update make command to build full release

* use quoted TAGS

`TAGS=bindata` => `TAGS="bindata"`

[skip ci] Updated translations via Crowdin

Upgrade gopkg.in/testfixtures.v2 (#4999)

[skip ci] Updated translations via Crowdin

Disable debug routes unless PPROF is enabled in configuration (#4995)

Fix #rrggbbaa color to rgba for better browser compatibility (#4990)

Detect charset and convert non UTF-8 files for display (#4950)

* Detect charset and convert non UTF-8 files for display

* Refactor and move function to correct module

* Revert unrelated changes

* More unrelated changes

* Duplicate content for small text to have better encoding detection

* Check if original content is valid before duplicating it

Fix layout of the topics editing form (#4971)

* Fix layout of the topic edit.

- made right-hand column wider so that it has enough space for "Done" button.

- fixed issue that jQuery's .show() breaks functionality of the Semantic UI Grid.

* Improve switching visibility of the topic edit

Changes to support old browsers that doesn't support 'flex' keyword.

- Removed style "display: none" from index.css so that
  the grid can be displayed without specifying new "display" style.

- Added style "display:none" to the grid element in HTML template
  as the initial style.

- In index.js, visibility of the grid element is changed by
  set "display:none" style to the element or removing it from the element.

make sure milestone listing is consistent across board (#4987)

Fix user menu item styling (#4985)

[skip ci] Updated translations via Crowdin

Adjust maintainers (#4979)

[skip ci] Updated translations via Crowdin

Fix preview when adding new code review comment (#4975)

Update Swagger API to match the return of /users/search (#4847)

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

ADD: Hint to function of docker version tags (#4967)

I wasn't sure about how tags work, maybe this is interesting for people with less knowledge of docker.

Fix #4877 to follow the OpenID Connect Audiences spec (#4878)

Signed-off-by: Gabriel Robertson <overminddl1@gmail.com>

[skip ci] Updated translations via Crowdin

Fix null pointer dereference in ParseCommitWithSignature (#4962)

[skip ci] Updated translations via Crowdin

Fix interpunction in English translation. (#4958)

[skip ci] Updated translations via Crowdin

fix url in discord webhook (#4953)

opening issues generates a webhook to discord that contains
a url to the gitea api. the message title in discord is therefore
referencing to the api instead of the issue itself.

Code review UI improvements and bugfixes (#4682)

* Code review UI improvements

* More fixes to dark theme

* Style fix

* Fix to allow add code review comments only on review files tab

* More readability dark style fixes

* Fix commenting on deleted files. Fixes #4752

* Fix line blame getting for multiple corner cases

Fix some issues with special chars in branch names (#3767)

Signed-off-by: Robin Durner <github@nubenum.de>

make sure to catch the right error so it is displayed as an error on the ui not a 500 (#4945)

fix(topics): don't redirect to expole page. (#4938)

[skip ci] Updated translations via Crowdin

doc: add Cloudron package (#4937)

Fixes #4936

Allow admin toggle forcing a password change for newly created users (#4563)

fix bug forget to remove Stopwatch when remove repository (#4928)

Force user to change password (#4489)

* redirect to login page after successfully activating account

* force users to change password if account was created by an admin

* force users to change password if account was created by an admin

* fixed build

* fixed build

* fix pending issues with translation and wrong routes

* make sure path check is safe

* remove unneccessary newline

* make sure users that don't have to view the form get redirected

* move route to use /settings prefix so as to make sure unauthenticated users can't view the page

* update as per @lafriks review

* add necessary comment

* remove unrelated changes

* support redirecting to location the user actually want to go to before being forced to change his/her password

* run make fmt

* added tests

* improve assertions

* add assertion

* fix copyright year

Signed-off-by: Lanre Adelowo <yo@lanre.wtf>

Fix bug when repo remained bare if multiple branches pushed (#4923)

feat(repo): support search repository by topic name (#4505)

* feat(repo): support search repository by topic name

[skip ci] Updated translations via Crowdin

Add/remove/list oauth providers via cli (#4591)