summaryrefslogtreecommitdiff
path: root/lib/api
Commit message (Collapse)AuthorAgeFilesLines
* Merge branch 'optimise-job-request' into 'master'Grzegorz Bizon2018-11-211-1/+2
|\ | | | | | | | | Use cached size when passing artifacts to Runner See merge request gitlab-org/gitlab-ce!23228
| * Use cached size when passing artifacts to Runneroptimise-job-requestKamil Trzciński2018-11-201-1/+2
| |
* | Change param name for consistencyHeinrich Lee Yu2018-11-191-2/+2
| |
* | Add group projects API options for including shared and subgroupsHeinrich Lee Yu2018-11-191-1/+13
|/
* Merge branch '41875-allow-pipelines-to-be-deleted-by-project-owners' into ↵Grzegorz Bizon2018-11-161-0/+15
|\ | | | | | | | | | | | | | | | | 'master' Resolve "Allow pipelines to be deleted by project owners" Closes #41875 See merge request gitlab-org/gitlab-ce!22988
| * Authorize DestroyPipelineService against pipelineMatija Čupić2018-11-131-1/+1
| |
| * Destroy pipeline in serviceMatija Čupić2018-11-121-4/+4
| | | | | | | | | | Move all logic for destroying a Pipeline into a service so it's easily reusable.
| * Implement MVC for Pipeline deletion APIMatija Čupić2018-11-121-0/+15
| |
* | Merge branch 'docs/rs-revert-api-version' into 'master'Evan Read2018-11-161-1/+1
|\ \ | | | | | | | | | | | | Revert API is going into 11.5, not 11.6. See merge request gitlab-org/gitlab-ce!23060
| * | Revert API is going into 11.5, not 11.6.docs/rs-revert-api-versionRobert Speicher2018-11-141-1/+1
| |/
* | Merge branch 'if-53347_fix_impersonation_tokens' into 'master'Stan Hu2018-11-152-3/+7
|\ \ | |/ |/| | | | | Display impersonation token value only after creation See merge request gitlab-org/gitlab-ce!22916
| * Display impersonation token value only after creationif-53347_fix_impersonation_tokensImre Farkas2018-11-122-3/+7
| | | | | | | | | | Since we migrated all PersonlAccessTokens to store only its hash in the DB, the token value can no longer be shown to the user.
* | Merge branch 'blackst0ne-add-discord-service' into 'master'Sean McGivern2018-11-121-0/+9
|\ \ | | | | | | | | | | | | | | | | | | Add Discord integration Closes #21635 See merge request gitlab-org/gitlab-ce!22684
| * | Add Discord integrationblackst0ne2018-10-301-0/+9
| | |
* | | Merge branch 'rs-cherry-pick-api' into 'master'Nick Thomas2018-11-091-3/+5
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Resolve possible cherry pick API race condition Closes #53773 See merge request gitlab-org/gitlab-ce!22946
| * | | Resolve possible cherry pick API race conditionrs-cherry-pick-apiRobert Speicher2018-11-091-3/+5
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously, we just fetched the latest commit for the given branch when presenting the resulting commit, but because something could have been committed to that branch between the time we cherry-picked and the time we render the result, the wrong commit could have been presented. Now, we fetch the commit object with the commit SHA returned by the commit change service, which should always be the correct commit. Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/53773
* | | Add revert to commits APIRobert Speicher2018-11-091-0/+34
|/ /
* | Merge branch 'fj-41213-api-update-submodule-commit' into 'master'Sean McGivern2018-11-072-0/+48
|\ \ | | | | | | | | | | | | | | | | | | Add endpoint to update a git submodule reference Closes #41213 See merge request gitlab-org/gitlab-ce!20949
| * | Add submodule update API endpointFrancisco Javier López2018-11-072-0/+48
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This new endpoint allow users to update a submodule's reference. The MR involves adding a new operation RPC operation in gitaly-proto (see gitlab-org/gitaly-proto!233) and change Gitaly to use this new version (see gitlab-org/gitaly!936). See gitlab-org/gitlab-ce!20949
* | | Merge branch 'dm-api-merge-requests-index-merged-at' into 'master'Sean McGivern2018-11-073-18/+18
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Expose {closed,merged}_{at,by} in merge requests API index Closes #52031 See merge request gitlab-org/gitlab-ce!22806
| * | | Expose {closed,merged}_{at,by} in merge requests API indexDouwe Maan2018-11-053-18/+18
| | | |
* | | | Merge branch 'master' into 'refactor-snippets-finder'refactor-snippets-finderDouwe Maan2018-11-062-3/+25
|\ \ \ \ | | |/ / | |/| | | | | | # Conflicts: # spec/models/project_spec.rb
| * | | Merge branch 'use_tokenauthenticatable_method_for_token_lookup' into 'master'Sean McGivern2018-11-051-3/+1
| |\ \ \ | | |/ / | |/| | | | | | | | | | Remove PersonalAccessTokensFinder#find_by method See merge request gitlab-org/gitlab-ce!22617
| | * | Remove PersonalAccessTokensFinder#find_by methoduse_tokenauthenticatable_method_for_token_lookupImre Farkas2018-11-051-3/+1
| | | | | | | | | | | | | | | | | | | | | | | | find_by_token is overriden by TokenAuthenticatable which can be easily missed or confused with #find_by(:token) defined by ActiveRecord. First step for safer usage is to remove #find_by.
| * | | add related merge request endpointHelmut Januschka2018-11-011-0/+24
| | | |
* | | | Rewrite SnippetsFinder to improve performanceYorick Peterse2018-11-051-1/+1
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This completely rewrites the SnippetsFinder class from the ground up in order to improve its performance. The old code was beyond salvaging. It was complex, included various Rails 5 workarounds, comments that shouldn't be necessary, and most important of all: it produced a really poorly performing database query. As a result, I opted for rewriting the finder from scratch, instead of trying to patch the existing code. Instead of trying to reuse as many existing methods as possible, I opted for defining new methods specifically meant for the SnippetsFinder. This requires some extra code here and there, but allows us to have much more control over the resulting SQL queries. It is these changes that then allow us to produce a _much_ more efficient query. To illustrate how bad the old query was, we will use my own snippets as an example. Currently I have 52 snippets, most of which are global ones. To retrieve these, you would run the following Ruby code: user = User.find_by(username: 'yorickpeterse') SnippetsFinder.new(user, author: user).execute On GitLab.com the resulting query will take between 10 and 15 seconds to run, producing the query plan found at https://explain.depesz.com/s/Y5IX. Apart from the long execution time, the total number of buffers (the sum of all shared hits) is around 185 GB, though the real number is probably (hopefully) much lower as I doubt simply summing these numbers produces the true total number of buffers used. The new query's plan can be found at https://explain.depesz.com/s/wHdN, and this query takes between 10 and 100-ish milliseconds to run. The total number of buffers used is only about 30 MB. Fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/52639
* | | Merge branch 'ce-7927-filter-none-any-weight-issues-api' into 'master'Sean McGivern2018-10-311-4/+13
|\ \ \ | |/ / |/| | | | | | | | Split CE and EE params in issues API See merge request gitlab-org/gitlab-ce!22689
| * | Extract EE params in issues API to separate moduleHeinrich Lee Yu2018-10-301-4/+13
| |/
* | Merge remote-tracking branch 'origin/master' into dev-masterDJ Mountney2018-10-297-9/+47
|\ \ | |/
| * Merge branch ↵Douwe Maan2018-10-261-0/+2
| |\ | | | | | | | | | | | | | | | | | | | | | | | | '42790-improve-feedback-for-internal-git-access-checks-timeouts' into 'master' Adds trace of each access check when git push times out Closes #42790 See merge request gitlab-org/gitlab-ce!22265
| | * Adds tracing messages for slow git pushesTiago Botelho2018-10-251-0/+2
| | | | | | | | | | | | | | | | | | Whenever a git push takes more than 50 seconds the user will receive a trace from each check performed along with their timings
| * | Feature/add license to project APIJ.D. Bean2018-10-262-5/+27
| | |
| * | Merge branch '52384-api-filter-assignee-none-any' into 'master'Sean McGivern2018-10-263-2/+17
| |\ \ | | | | | | | | | | | | | | | | | | | | | | | | Resolve "Filter by `None`/`Any` for assignee_id in issues/mrs API" Closes #52384 See merge request gitlab-org/gitlab-ce!22598
| | * | Support lowercase none / anyHeinrich Lee Yu2018-10-261-1/+1
| | | |
| | * | Refactor api validator to separate classHeinrich Lee Yu2018-10-263-8/+15
| | | |
| | * | Apply similar change to MRs APIHeinrich Lee Yu2018-10-262-2/+6
| | | |
| | * | Issues API: Add None/Any option to assignee_idHeinrich Lee Yu2018-10-261-1/+5
| | |/
| * | Support backward compatibility when introduce new failure reasonShinya Maeda2018-10-261-2/+1
| |/
* | Merge branch 'sh-fix-wiki-security-issue-53072' into 'master'Jan Provaznik2018-10-292-2/+17
|\ \ | |/ |/| | | | | [master] Validate Wiki attachments are valid temporary files See merge request gitlab/gitlabhq!2568
| * Validate Wiki attachments are valid temporary filesStan Hu2018-10-232-2/+17
| | | | | | | | | | | | | | | | | | | | A malicious attacker could craft a request to read arbitrary files on the system. This change adds a Grape validation to ensure that the tempfile parameter delivered by the Rack multipart uploader is a Tempfile type to prevent users from being able to specify arbitrary filenames. Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/53072
* | Merge branch '52559-applications-api-get-delete' into 'master'Rémy Coutable2018-10-222-0/+18
|\ \ | |/ |/| | | | | | | | | Add Applications API endpoints for listing and deleting entries. Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/52559 See merge request https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/22296
| * Use application finder for Doorkeeper ApplicationsJB Vasseur2018-10-151-6/+3
| |
| * Do not return secret from GET /applications !22296JB Vasseur2018-10-121-1/+1
| |
| * disable CodeReuse/ActiveRecordJB Vasseur2018-10-121-0/+2
| |
| * Improve call for retrieving all applications !22296JB Vasseur2018-10-121-1/+1
| |
| * Support GET /applications and DELETE /applications/:id endpoints #52559JB Vasseur2018-10-111-0/+17
| |
| * Expose id and name attributes to Applications APIJB Vasseur2018-10-111-0/+2
| |
* | Rename "scheduled" label/badge of delayed jobs to "delayed"Shinya Maeda2018-10-191-1/+1
| |
* | Merge branch 'bvl-remove-forked-project-link' into 'master'Sean McGivern2018-10-191-1/+1
|\ \ | | | | | | | | | | | | | | | | | | Remove ForkedProjectLink model Closes #38883 See merge request gitlab-org/gitlab-ce!22226
| * | Remove the `ForkedProjectLink` modelBob Van Landuyt2018-10-191-1/+1
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | This removes the `ForkedProjectLink` model that has been replaced by the `ForkNetworkMember` and `ForkNetwork` combination. All existing relations have been adjusted to use these new models. The `forked_project_link` table has been dropped. The "Forks" count on the admin dashboard has been updated to count all `ForkNetworkMember` rows and deduct the number of `ForkNetwork` rows. This is because now the "root-project" of a fork network also has a `ForkNetworkMember` row. This count could become inaccurate when the root of a fork network is deleted.