aboutsummaryrefslogtreecommitdiffstats
path: root/modules
Commit message (Collapse)AuthorAgeFilesLines
* Ensure Written is set in GZIP ProxyResponseWriter (#9018) (#9026)release/v1.9zeripath2019-11-151-10/+43
| | | | | | | | 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.
* Add Close() method to gogitRepository (#8901) (#8958)zeripath2019-11-1331-2/+122
| | | | | | | | | | | 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)mrsdizzie2019-11-131-2/+2
| | | | Small fix for test on 1.9 since #8938 can't be easily back ported to this branch.
* Update Github migration test (#8897)guillep2k2019-11-091-6/+0
| | | | | | | 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)Lunny Xiao2019-11-021-0/+5
|
* Allow to merge if file path contains " or \ (#8629) (#8772)zeripath2019-11-011-2/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * 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>
* Use AppSubUrl for more redirections (#8647) (#8652)zeripath2019-10-231-1/+1
| | | | | | 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)John Olheiser2019-10-231-1/+1
| | | Signed-off-by: jolheiser <john.olheiser@gmail.com>
* Fix #8582 by handling empty repos (#8587) (#8593)65432019-10-192-0/+19
| | | | | | | | | * Fix #8582 by handling empty repos Signed-off-by: Jonas Franz <info@jonasfranz.software> * Fix tests Signed-off-by: Jonas Franz <info@jonasfranz.software>
* IsBranchExist: return false if provided name is empty (#8485) (#8492)zeripath2019-10-131-2/+8
| | | | | * IsBranchExist: return false if provided name is empty * Ensure that the reference returned is actually of a valid type
* Restore functionality for early gits (#7775) (#8476)zeripath2019-10-137-50/+137
| | | | | | | | | | | | | | | | | | | | | | | | * 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
* Ensure Request Body Readers are closed in LFS server (#8454) (#8459)zeripath2019-10-112-5/+15
|
* fix bug when migrating a private repository (#7917) (#8403)65432019-10-071-1/+12
|
* feat: highlight issue references with : (#8101) (#8404)65432019-10-062-2/+9
| | | | | | | | | | * 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 data race (#8206)Mura Li2019-09-172-17/+43
| | | | | | | | | | | | | | * 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 pull merge 500 error caused by git-fetch breaking behaviors (#8194)Mura Li2019-09-161-1/+1
|
* [backport/v1.9] Fix reading git notes from nested trees (#8189)Filip Navara2019-09-1523-15/+59
| | | | | | | | | | | | | * 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 Go 1.13 private repository go get issue (#8100)Rutger Broekhoff2019-09-062-3/+20
| | | | | | | | | | * 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)guillep2k2019-09-042-4/+17
| | | | | | * Strict name matching for Repository.GetTagID() * Add test for GetTagID()
* Avoid ambiguity of branch/directory names for the git-diff-tree command ↵Mura Li2019-09-031-2/+1
| | | | (#8066) (#8070)
* Backport for #7945 (#7994)Richard Mahn2019-08-271-1/+1
|
* Abort syncrhonization from LDAP source if there is some error. (#7965)David Svantesson2019-08-251-5/+5
| | | | Signed-off-by: David Svantesson <davidsvantesson@gmail.com> (cherry picked from commit b2d23a138942179d62f9bffdb3df272e9300e35b)
* Attempt to fix hook problem (#7856)zeripath2019-08-141-1/+3
|
* Do not fetch all refs (#7837)Mura Li2019-08-131-1/+1
| | | Which would unnecessarily slow down the pull compare operation.
* Fix duplicate call of webhook (#7821) (#7824)Antoine GIRARD2019-08-124-112/+0
|
* Be more strict with git arguments (#7715) (#7762)zeripath2019-08-0512-20/+41
| | | | | * Be more strict with git arguments * fix-up commit test * use bindings for branch name
* Backport: skip non-regular files (e.g. submodules) on repo indexing (#7717)guillep2k2019-08-021-0/+5
| | | | | | * Backport: skip non-regular files (e.g. submodules) on repo indexing * Include "executable" files in the index, as they are not necessarily binary
* Correct wrong datetime format for git (#7689) (#7690)zeripath2019-07-312-2/+2
|
* fix bug on migrating milestone from github (#7665) (#7666)Lunny Xiao2019-07-291-1/+1
|
* Fix bug create/edit wiki pages when code master branch protected (#7580) (#7623)Lunny Xiao2019-07-261-1/+7
| | | | | * fix bug create/edit wiki pages when code master branch protected * improve FullPushingEnvironment function
* Fix panic on push at #7611 (#7615) (#7618)zeripath2019-07-251-1/+1
| | | | | | | * Fix panic in #7611 Use pr.IssueID instead of pr.Issue.ID as Issue may not be loaded and is unnecessary * Only fetch the head branch from the remote
* Fix empty commits now showing in repo overview (#7521) (#7563)silverwind2019-07-227-13/+24
| | | | | | | * Fix empty commits now showing in repo overview * add test * make fmt
* Fix markdown invoke sequence (#7513) (#7560)Lunny Xiao2019-07-221-1/+1
|
* remove duplicated webhook trigger (#7511) (#7516)Lunny Xiao2019-07-181-1/+0
|
* Fix migration tests since #7 fixed (#7375) (#7381)Lunny Xiao2019-07-081-2/+4
| | | | | * fix migration tests since #7 fixed * fix test time
* Fix typo in PR migration check (#7368)mrsdizzie2019-07-061-1/+1
|
* Make captcha and password optional for external accounts (#6606)AJ ONeal2019-07-062-1/+6
|
* Detect migrating batch size (#7353)Lunny Xiao2019-07-063-17/+67
| | | | | | | | * Make migrating batch size as configurable * detect different table batch insert size and remove config item * remove unrelated changes
* Use commit graph files for listing pages (#7314)Filip Navara2019-07-023-12/+73
| | | | | | | | | | | | | | | | | | | | | | | | * Experimental support for git commit graph files and bloom filter index Signed-off-by: Filip Navara <filip.navara@gmail.com> * Force vendor of commitgraph Signed-off-by: Filip Navara <filip.navara@gmail.com> * Remove bloom filter experiment and debug prints * Remove old code for building commit graphs * Remove unused function * Remove mmap usage * gofmt * sort vendor/modules.txt * Add copyright header and log commit-graph error
* Fix bug conflict between SyncReleasesWithTags and InsertReleases (#7337)Lunny Xiao2019-07-013-10/+16
| | | | | | | | * fix bug conflict between SyncReleasesWithTags and InsertReleases * fix tests * fix fmt
* #6946 Run hooks on merge/edit and cope with protected branches (#6961)zeripath2019-07-012-2/+14
| | | | | | | | | | | | | | | | | | * Fix #6946 by checking PullRequest ID on pushing * Ensure we have the owner name, the pr attributes and the the issue * Fix TestSearchRepo by waiting till indexing is done * Update integrations/repo_search_test.go * changes as per @mrsdizzie * missing comma * Spelling mistake * Fix full pushing environment
* Add commit statuses reports on pull request view (#6845)Lunny Xiao2019-06-301-0/+39
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Add commit statuses reports on pull view * Add some translations * improve the UI * fix fmt * fix tests * add a new test git repo to fix tests * fix bug when headRepo or headBranch missing * fix tests * fix tests * fix consistency * fix tests * fix tests * change the test repo * fix tests * fix tests * fix migration * keep db size consistency * fix translation * change commit hash status table unique index * remove unused table * use char instead varchar * make hashCommitStatusContext private * split merge section with status check on pull view ui * fix tests; fix arc-green theme on pull ui
* Fixes #7292 - API File Contents bug (#7301)Richard Mahn2019-06-297-81/+372
|
* Fixes #7152 - Allow create/update/delete message to be empty, use default ↵Richard Mahn2019-06-292-1/+6
| | | | | | | | | | message (#7324) * Fixes #7152 - Allow create/update/delete message to be empty, use default message * Linting fix * Fix to delete integration tests
* Use batch insert on migrating repository to make the process faster (#7050)Lunny Xiao2019-06-296-187/+259
| | | | | | | | | | * Use batch insert on migrating repository to make the process faster * fix lint * fix tests * fix comments
* add commitgraph support global default true when git version >= 2.18 (#7313)Lunny Xiao2019-06-291-0/+12
|
* Fixes #7238 - Annotated tag commit ID incorrect (#7321)Richard Mahn2019-06-291-3/+4
| | | | | | * Fixes #7238 - Annotated tag commit ID incorrect * Fixes #7238 - Annotated tag commit ID incorrect
* Monitor all git commands; move blame to git package and replace git as a ↵Lunny Xiao2019-06-2625-12/+959
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | variable (#6864) * monitor all git commands; move blame to git package and replace git as a variable * use git command but not other commands * fix build * move exec.Command to git.NewCommand * fix fmt * remove unrelated changes * remove unrelated changes * refactor IsEmpty and add tests * fix tests * fix tests * fix tests * fix tests * remove gitLogger * fix fmt * fix isEmpty * fix lint * fix tests
* fix API link header (#7298)Lunny Xiao2019-06-262-7/+85
|
* Fix #732: Add LFS objects to base repository on merging (#7082)zeripath2019-06-222-0/+565
| | | | On merge we walk the merge history and ensure that all lfs objects pointed to in the history are added to the base repository. This switches from relying on having git-lfs installed on the server, (and in fact .gitattributes being correctly installed.)