| Commit message (Collapse) | Author | Age | Files | Lines |
|---|
| |
|
|
| |
It's used only if merge_request_iid param is provided
|
| |\
| |
| |
| |
| |
| |
| | |
Use NotesFinder to fetch notes on API and Controllers
Closes #52123
See merge request gitlab-org/gitlab-ce!31300
|
| | | |
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Remove project from NotesFinder constructor
Add project parameter to specs
Also look for methods in private scope
Fix specs to match new NotesFinder constructor
|
| |\ \
| | |
| | |
| | |
| | | |
Fix bug when moving batches of items to the end
See merge request gitlab-org/gitlab-ce!31351
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | | |
Starts from START_POSITION when there are no existing
positions.
Also improves the test to actually test the behavior
|
| |/ / |
|
| |/ |
|
| |
|
|
|
|
|
|
|
| |
1. The output isn't great. It can be hard to find hotspots and, even
when you do find them, to find why those are hotspots.
2. It uses some jQuery-specific frontend code which we can remove now
that we don't have this any more.
3. It's only possible to profile the initial request, not any subsequent
AJAX requests.
|
| |\ |
|
| | |\
| | |
| | |
| | |
| | |
| | |
| | | |
Don't display badges when builds are restricted
Closes #2864
See merge request gitlab/gitlabhq!3175
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Badges were leaked to unauthorized users even when Public Builds
project setting is disabled.
Added guard clause to the controller to check if user can read
build.
|
| | |\ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Drop feature to take ownership of a trigger token
Closes #2868
See merge request gitlab/gitlabhq!3198
|
| | | |/
| | |
| | |
| | |
| | |
| | |
| | | |
Removing API and frontend interactions that allowed
users to take ownership of a trigger token.
Removed mentions from the documentation.
|
| | |\ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
MR pipeline permissions
Closes #2871
See merge request gitlab/gitlabhq!3204
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
MergeRequest#all_pipelines
MergeRequest#all_pipelines fetches Ci::Pipeline records from the source
project, so we should specifically check that project for permissions.
This was already happening for intra-project merge requests, but in the
event that the target and source projects both have private builds, we
should ensure that the project permissions are respected.
|
| | |\ \ \
| | | | |
| | | | |
| | | | |
| | | | | |
Queries for Upload should be scoped by model
See merge request gitlab/gitlabhq!3229
|
| | | | |/
| | |/| |
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
Added usage ping counter when the user makes
a search through the navbar search component.
|
| |\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | | |
Separate private methods under the keyword on projects controller
See merge request gitlab-org/gitlab-ce!31223
|
| | | | | | |
|
| |/ / / /
| | | |
| | | |
| | | |
| | | | |
Adds Job specific variables to facilitate specifying variables when
running manual jobs.
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
'master'"
This reverts merge request !29733
|
| |\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Use NotesFinder to fetch notes in module IssuableActions
Closes #52123
See merge request gitlab-org/gitlab-ce!29733
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Add spec for concern IssuableActions
Add shared samples for discussions endpoint
Add schema validations for discussions
Fix rubocop style issue
Make target assignable
Use new possibility to provide target
|
| |\ \ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Remove code related to object hierarchy and MySQL
Closes #65056 and #65055
See merge request gitlab-org/gitlab-ce!31095
|
| | | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
These are not required because MySQL is not
supported anymore
|
| |\ \ \ \ \ \
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Find build by sha from ref
Closes #64534 and #45697
See merge request gitlab-org/gitlab-ce!30843
|
| | | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
* Reword Project#latest_successful_build_for to
Project#latest_successful_build_for_ref
* Reword Ci::Pipeline#latest_successful_for to
Ci::Pipeline#latest_successful_build_for_ref
|
| | | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Adds ability to find builds by sha when only specifying a ref.
|
| | |_|/ / / /
|/| | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
By not triggering the callback:
- ActiveSession lookup keys are not cleaned
- Devise also misses its hook related to session cleanup
|
| |\ \ \ \ \ \
| |_|/ / / /
|/| | | | |
| | | | | |
| | | | | | |
Return an ETag header for the archive endpoint
See merge request gitlab-org/gitlab-ce!30581
|
| | | |_|/ /
| |/| | |
| | | | |
| | | | | |
We use the relative path of the archive to check for archive staleness.
|
| |\ \ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Enable Rubocop Performance/InefficientHashSearch
Closes #64975
See merge request gitlab-org/gitlab-ce!31117
|
| | | |_|/ /
| |/| | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
When used with a Hash, `.keys.include?` is bad because:
1. It performs a O(n) search instead of the efficient `.has_key?`
2. It clones all keys into separate array.
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/64975
|
| |/ / / /
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
* Limits raw requests to 300 per minute and per raw path.
* Add a new attribute to ApplicationSettings so user can change this
value on their instance.
* Uses Gitlab::ActionRateLimiter to limit the raw requests.
* Add a new method into ActionRateLimiter to log the event into auth.log
Related to https://gitlab.com/gitlab-org/gitlab-ce/issues/48717
|
| |\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | | |
Remove OAuth GitHub CI/CD project code paths
See merge request gitlab-org/gitlab-ce!30716
|
| | | | | | |
|
| |\ \ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Default manual_sorting feature flag to on
See merge request gitlab-org/gitlab-ce!30788
|
| | |/ / / / |
|
| | |_|/ /
|/| | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
This removes the create_wiki permission check from the history
controller, allowing show and history to have the same level of
permissions.
Fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/29528
|
| |\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | | |
CE Port: Log impersonation actions in audit log
See merge request gitlab-org/gitlab-ce!31039
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
This change adds audit logs for user impersonation
when an admin starts/stops impersonating
another user.
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
It seems that we missed the backward compatibility support
for profiles in the existing folder.
This commit also fixes some specs to be idempotent
and work in a temporary directory which not always
seems to be the case.
This commit also brings the profile_spec.rb which seems
to be missing.
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Add cr remarks
Add cr remarks
|
| |\ \ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Add a group setting to allow Maintainers to create sub-groups
See merge request gitlab-org/gitlab-ce!29718
|
| | | | | | |
| | | | | |
| | | | | |
| | | | | | |
For both groups_controller and admin/groups_controller
|
| |\ \ \ \ \ \
| |/ / / / /
|/| | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Remove `if:` argument in before_action and alike when `only:` is also used
Closes #55564
See merge request gitlab-org/gitlab-ce!24412
|
| | | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Closes #55564
This is first discovered in #54739 (comment 122609857) that if both if: and only:
are used in a before_action or after_action or alike, if: is completely ignored.
|
| | | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
The actions tracked in the web IDE are:
- creation of commits
- creation of merge requests
- projects loaded
|
