| Commit message (Collapse) | Author | Age | Files | Lines |
|\
| |
| |
| |
| | |
Use cached size when passing artifacts to Runner
See merge request gitlab-org/gitlab-ce!23228
|
| | |
|
| | |
|
|/ |
|
|\
| |
| |
| |
| |
| |
| |
| |
| | |
'master'
Resolve "Allow pipelines to be deleted by project owners"
Closes #41875
See merge request gitlab-org/gitlab-ce!22988
|
| | |
|
| |
| |
| |
| |
| | |
Move all logic for destroying a Pipeline into a service so it's easily
reusable.
|
| | |
|
|\ \
| | |
| | |
| | |
| | | |
Revert API is going into 11.5, not 11.6.
See merge request gitlab-org/gitlab-ce!23060
|
| |/ |
|
|\ \
| |/
|/|
| |
| | |
Display impersonation token value only after creation
See merge request gitlab-org/gitlab-ce!22916
|
| |
| |
| |
| |
| | |
Since we migrated all PersonlAccessTokens to store only its hash in the
DB, the token value can no longer be shown to the user.
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | | |
Add Discord integration
Closes #21635
See merge request gitlab-org/gitlab-ce!22684
|
| | | |
|
|\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Resolve possible cherry pick API race condition
Closes #53773
See merge request gitlab-org/gitlab-ce!22946
|
| | |/
| |/|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
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 endpoint to update a git submodule reference
Closes #41213
See merge request gitlab-org/gitlab-ce!20949
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
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
|
|\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Expose {closed,merged}_{at,by} in merge requests API index
Closes #52031
See merge request gitlab-org/gitlab-ce!22806
|
| | | | |
|
|\ \ \ \
| | |/ /
| |/| |
| | | | |
# Conflicts:
# spec/models/project_spec.rb
|
| |\ \ \
| | |/ /
| |/| |
| | | |
| | | | |
Remove PersonalAccessTokensFinder#find_by method
See merge request gitlab-org/gitlab-ce!22617
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
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.
|
| | | | |
|
|/ / /
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
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
|
|\ \ \
| |/ /
|/| |
| | |
| | | |
Split CE and EE params in issues API
See merge request gitlab-org/gitlab-ce!22689
|
| |/ |
|
|\ \
| |/ |
|
| |\
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
'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
|
| | |
| | |
| | |
| | |
| | |
| | | |
Whenever a git push takes more than 50 seconds the user
will receive a trace from each check performed along with
their timings
|
| | | |
|
| |\ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Resolve "Filter by `None`/`Any` for assignee_id in issues/mrs API"
Closes #52384
See merge request gitlab-org/gitlab-ce!22598
|
| | | | |
|
| | | | |
|
| | | | |
|
| | |/ |
|
| |/ |
|
|\ \
| |/
|/|
| |
| | |
[master] Validate Wiki attachments are valid temporary files
See merge request gitlab/gitlabhq!2568
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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
|
|\ \
| |/
|/|
| |
| |
| |
| | |
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
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | | |
Remove ForkedProjectLink model
Closes #38883
See merge request gitlab-org/gitlab-ce!22226
|
| |/
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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.
|