Age | Commit message (Collapse) | Author |
|
Update Helm to 2.12.2 to address Helm client vulnerability
Closes #56417
See merge request gitlab-org/gitlab-ce!24418
|
|
'25569-changing-wording-to-delete-when-referring-to-removing-a-branch' into 'master'
Resolve "Changing wording to "Delete" when referring to removing a branch"
Closes #25569
See merge request gitlab-org/gitlab-ce!23966
|
|
Fix bug that caused Suggestion Markdown toolbar button to insert snippet with leading +/-/<space>
See merge request gitlab-org/gitlab-ce!24568
|
|
Add domain field into Clusters::Cluster
See merge request gitlab-org/gitlab-ce!24368
|
|
This is to match `git branch -D <branchname>`
|
|
Adds inter-service OpenTracing propagation
See merge request gitlab-org/gitlab-ce!24239
|
|
section
|
|
This is the 1st step for moving Auto DevOps domain into cluster
settings, whether is project or group.
Related to https://gitlab.com/gitlab-org/gitlab-ce/issues/52363
|
|
'master'
Removes factories on Cluster background migration
See merge request gitlab-org/gitlab-ce!24536
|
|
Hashed Storage: `AfterRenameService` was not renaming Pages or Uploads folder on legacy storage
Closes #56636
See merge request gitlab-org/gitlab-ce!24526
|
|
This change allows the GitLab rails and sidekiq components to receive
tracing spans from upstream services such as Workhorse and pass these
spans on to downstream services including Gitaly and Sidekiq.
This change will also emit traces for incoming and outgoing requests
using the propagated trace information. This will allow operators and
engineers to view traces across the Workhorse, GitLab Rails, Sidekiq and
Gitaly components.
Additional intra-service instrumentation will be added in future
changes.
|
|
Before, diff file `higlighted_diff_lines`/`parallel_diff_lines` and
diff discussion `truncated_diff_lines` were inconsistent: `text` and
`rich_text` on the latter included the leading +/-/<space> character,
like on the backend, while the former had no `text` and its `rich_text`
had dropped this char.
This resulted in a bug when the suggestions feature expected these diff
line objects to be identical in format and thus interchangeable, which
was not the case.
|
|
Changed the Caching of User Avatars to be public and to 5 minutes
See merge request gitlab-org/gitlab-ce!24546
|
|
'55966-when-ref-is-ambiguous-createpipelineservice-raises-an-error' into 'master'
Resolve "When ref is ambiguous, `CreatePipelineService` raises an error"
Closes #55966
See merge request gitlab-org/gitlab-ce!24437
|
|
Make favicon tests more fault resistent
Closes #50527
See merge request gitlab-org/gitlab-ce!22686
|
|
'Populate cluster kubernetes namespace' was using factories for their
specs. According to our documentation (see spec/migrations/readme.md),
we should use table helper to create a temproary ActiveRecord::Base
derived model for a table.
|
|
|
|
'master'
Resolve "Related merge requests in issue design - Restyle"
Closes #47007
See merge request gitlab-org/gitlab-ce!24270
|
|
Resolve "Limit sidekiq logging based on argument size"
Closes #56547
See merge request gitlab-org/gitlab-ce!24493
|
|
Fix typos in dev & test docu
See merge request gitlab-org/gitlab-ce!24539
|
|
Resolve "Redirect projects/:id to project page"
Closes #53671
See merge request gitlab-org/gitlab-ce!24467
|
|
We still rely on the Dirty API for project rename (before/after) values,
but we don't access the dirty api from the service class anymore.
The previous value is now part of the initialization, which makes it
easier to test and the behavior is clearer.
The same was done with the `rename_repo` on the Storage classes, we now
provide before and after values as part of the method signature.
|
|
Also removed unnecessary comment
|
|
When logging arguments from Sidekiq to JSON, restrict the size of `args`
to 10 KB (when converted to JSON). This is to avoid blowing up with
excessively large job payloads.
|
|
'53856-changing-group-visibility-does-not-re-enable-save-button' into 'master'
Resolve "Changing group visibility does not re-enable Save button"
Closes #53856
See merge request gitlab-org/gitlab-ce!23022
|
|
Add CSS & JS global flags to represent browser and platform
Closes #50013
See merge request gitlab-org/gitlab-ce!24017
|
|
Fix 404s with snippet uploads in object storage
Closes #52595
See merge request gitlab-org/gitlab-ce!24550
|
|
|
|
|
|
By visiting `projects/:id` you will be redirected to project page with
path in it.
projects/123 => foo/bar
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
|
|
Previously, an HTTP request for
`/uploads/-/system/personal_snippet/:snippet_id/:hash/:filename` would
look for an uploader of `PersonalFileUploader` class and use
`PersonalFileUploader#upload_paths` to search the datbase for one of the
following paths:
1. `:hash/:filename`
2. `uploads/-/system/personal_snippet/:id/:hash/:filename`
However, when the upload were stored in object storage,
`PersonalFileUploader#store_dirs` stored the path as:
`personal_snippet/:snippet_id/:hash`
The extraneous `uploads/-/system` prefix prevented the path from being
matched, and uploads in object storage would return a 404 error. Uploads
in local storage would work fine.
To fix this, we set the `#base_dir` properly so that `#upload_paths`
generates the right value for object storage. Note that this also makes
`#store_dirs` do the right thing in `FileUploader`.
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/52595
|
|
Most browsers do not fire the "input" event for checkboxes or radios.
Adds a "change" listener to properly trigger these DirtySubmit updates.
|
|
|
|
|
|
It seems like every new version of Chrome renders the favicon canvas
differently. This reduces the problems with it by creating a more fault
tolerant check, which compares the color values in the pixels and is
okay if they are within a certain threshold.
|
|
Resolve "Don't check confidential issues for spam"
Closes #56371
See merge request gitlab-org/gitlab-ce!24453
|
|
|
|
During a previous refactor on project model, code related to the
hashed storage was extracted into AfterRenameService, see
4b9c17f196bab6075563f62d01f9db65c1a0515c.
The "path_before" was changed from using `previous_changes['path']` to
`path_was`. They are not equivalent. `path_was` exists reliably only
*before* persisting to the database. After database persistence is
confirmed, the value is moved to `previous_changes[:attribute_name]`.
Because the repository/attachments rename or storage upgrade happens
after it was persisted to the database, we were in fact not informing
the right parameters (and therefore not doing what it was supposed to).
|
|
This removes the Protect pipeline chain step and assigns the protected
attribute in the Populate step instead.
|
|
|
|
Eliminate N+1 queries in /api/groups/:id
Closes #49845
See merge request gitlab-org/gitlab-ce!24513
|
|
|
|
Set ActionController raise_on_unfiltered_parameters to true
See merge request gitlab-org/gitlab-ce!24443
|
|
Bump the katex package to 0.10.0
See merge request gitlab-org/gitlab-ce!24478
|
|
In https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/15475/diffs, a
significant amount of work went into eliminating N+1 queries in the
/api/groups/:id/projects endpoint. We can reuse the
`Entities::Project.prepare_relation` call on the projects.
In a group with 2,573 projects on GitLab.com, this change significantly
improves performance:
* 18019 SQL queries down to 21
* Time spent in DB: 70 s down to 384 ms
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/49845
|
|
Resolve "Remove `error_tracking` feature flag"
Closes #56213
See merge request gitlab-org/gitlab-ce!24306
|
|
|
|
Improvements for spec migration readme
See merge request gitlab-org/gitlab-ce!24440
|
|
Refer to https://github.com/KaTeX/KaTeX/pull/1700 for additional details.
|
|
This moves setting the protected attribute of a pipeline to a separate
pipeline chain step in order to perform the assignment after validation.
|