| Commit message (Collapse) | Author | Age | Files | Lines |
|---|
| |\
| |
| |
| |
| |
| |
| | |
Fix Peek on Puma
Closes #66528
See merge request gitlab-org/gitlab-ce!32213
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Previously, we called the `peek_enabled?` method like so:
prepend_before_action :set_peek_request_id, if: :peek_enabled?
Now we don't have a `set_peek_request_id` method, so we don't need that
line. However, the `peek_enabled?` part had a side-effect: it would also
populate the request store cache for whether the performance bar was
enabled for the current request or not.
This commit makes that side-effect explicit, and replaces all uses of
`peek_enabled?` with the more explicit
`Gitlab::PerformanceBar.enabled_for_request?`. There is one spec that
still sets `SafeRequestStore[:peek_enabled]` directly, because it is
contrasting behaviour with and without a request store enabled.
The upshot is:
1. We still set the value in one place. We make it more explicit that
that's what we're doing.
2. Reading that value uses a consistent method so it's easier to find in
future.
|
| | |
| |
| |
| | |
All avatars now visible in commit trailers.
|
| | |
| |
| |
| |
| |
| |
| | |
- Use "results" instead of "blobs", "wiki blobs", "snippet blobs"
- Use "comments" instead of "notes"
- Use correct pluralization
- Don't add "1 - 10 of" if there's only one page
|
| |\ \ |
|
| | |\ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Use image proxy to mitigate stealing ip addresses
Closes #2812
See merge request gitlab/gitlabhq!2926
|
| | | |/
| | |
| | |
| | |
| | |
| | | |
User images and videos will get proxied through
the Camo server in order to keep malicious
sites from collecting the IP address of users.
|
| | |\ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
into 'master'
Fix HTML injection for label description
See merge request gitlab/gitlabhq!3250
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Add changelog entry
Add spec
|
| | |\ \ \
| | | | |
| | | | |
| | | | |
| | | | | |
Require a captcha after unique failed logins from the same IP
See merge request gitlab/gitlabhq!3270
|
| | | |/ /
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Add method to store session ids by ip
Add new specs for storing session ids
Add cleaning up records after login
Add retrieving anonymous sessions
Add login recaptcha setting
Add new setting to sessions controller
Add conditions for showing captcha
Add sessions controller specs
Add admin settings specs for login protection
Add new settings to api
Add stub to devise spec
Add new translation key
Add cr remarks
Rename class call
Add cr remarks
Change if-clause for consistency
Add cr remarks
Add code review remarks
Refactor AnonymousSession class
Add changelog entry
Move AnonymousSession class to lib
Move store unauthenticated sessions to sessions controller
Move link to recaptcha info
Regenerate text file
Improve copy on the spam page
Change action filter for storing anonymous sessions
Fix rubocop offences
Add code review remarks
|
| | |\ \ \
| | |_|/
| |/| |
| | | |
| | | | |
Prevent disclosure of merge request id via email
See merge request gitlab/gitlabhq!3313
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
Do not disclosure merge request id via email for unauthorized users
when closing issues.
|
| | | | | |
|
| |\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | | |
Change the way totalNotes is calculated
See merge request gitlab-org/gitlab-ce!32191
|
| | |/ / /
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
totalNotes is only used to prerender a number of
skeleton containers until real notes are loaded
issuable.discussions makes multiple requests, so
too expensive for this
This commit uses mere notes for this and sends
actual totalNotes number if it's less than 10;
otherwise it sends 10 - it allows us to avoid
bunch of skeleton prerenderings, which are not
necessary since they doesn't fit into the whole
screen and disappear quite fast
|
| |\ \ \ \
| |/ / /
|/| | |
| | | |
| | | |
| | | |
| | | | |
Add Issue and Merge Request titles to Todo items
Closes #63488
See merge request gitlab-org/gitlab-ce!30435
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Only displays the todo body if the todo has a note.
This is to avoid redundant Issue or Merge Request titles
displayed both in the Todo title and body.
|
| |\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
'master'
Fix issuable sidebar icon of notification disabled
See merge request gitlab-org/gitlab-ce!32134
|
| | | | | |
| | | | |
| | | | |
| | | | | |
by adding the proper notification icon
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Some pipeline status icon tooltips were showing
"Commit: ..." which customers found to be
misleading since it was not the commit that was
failing but the pipeline.
We are changing all status icon tooltips to say
"Pipeline: ..." instead of "Commit: ..." now.
|
| | |/ / /
|/| | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Oauth2 tokens are causing issues with mirroring
repos, because it effectively limits the number
of repos you can mirror. Personal Access
Tokens do not have this problem.
This change removes the OAuth2 option from
the import page for CI/CD only, and only
provides the personal access token form.
|
| |/ / /
| | |
| | |
| | | |
Added snowplow event tracking to Edit buttons
|
| | | |
| | |
| | |
| | |
| | |
| | | |
- Adds UI to configure in group and project settings
- Removes notification configuration for users when
disabled at group or project level
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
CE-specific changes for:
https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/15129
Co-Authored-By: Alex Kalderimis <akalderimis@gitlab.com>
Co-Authored-By: Luke Duncalfe <lduncalfe@eml.cc>
|
| | | |
| | |
| | |
| | |
| | | |
This link is shown when a user tries to login with an unconfirmed
email address and the grace period has expired
|
| | | |
| | |
| | |
| | |
| | | |
This introduces several changes, but these are all just ported from the
EE project.
|
| | | |
| | |
| | |
| | |
| | |
| | | |
into 'master'"
This reverts merge request !30808
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
This adds a notification to let users know of our updated privacy
policy.
Users can dismiss the notification either by following the link or
closing the notification via an "x" icon.
|
| | | |
| | |
| | |
| | |
| | | |
Creates `refresh_counts` module to dynamically fetch
and load data based on attributes of HAML elements.
|
| | | |
| | |
| | |
| | |
| | |
| | | |
Querying all counts for the different search results in the same request
led to timeouts, so we now only calculate the count for the *current*
search results, and request the others in separate asynchronous calls.
|
| |\ \ \
| | | |
| | | |
| | | |
| | | | |
Remove KubernetesService integration
See merge request gitlab-org/gitlab-ce!31365
|
| | |/ /
| | |
| | |
| | |
| | |
| | | |
Creating new records has been disabled, and all
existing records been migrated to clusters as of
https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/28534
|
| | | | |
|
| | | | |
|
| | | | |
|
| |/ / |
|
| | |
| |
| |
| |
| | |
`allow_local_requests_for_hooks_and_services` was renamed to
`allow_local_requests_for_web_hooks_and_services`.
|
| |/
|
|
|
|
|
| |
This MR adds new application setting to network section
`allow_local_requests_from_system_hooks`. Prior to this change
system hooks were allowed to do local network requests by default
and we are adding an ability for admins to control it.
|
| |\
| |
| |
| |
| |
| |
| |
| |
| | |
'63547-add-system-notes-for-when-a-zoom-call-was-added-removed-from-an-issue' into 'master'
Resolve "Add system notes for when a zoom call was added/removed from an issue"
Closes #63547
See merge request gitlab-org/gitlab-ce!30857
|
| | |
| |
| |
| |
| | |
Add a zoom link added / removed system note when a zoom link is being
added / removed to the issue description.
|
| | | |
|
| |\ \
| | |
| | |
| | |
| | |
| | |
| | | |
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
|
| |/ /
| |
| |
| | |
Without this change, we get a NoMethodError on nil
|
| | |
| |
| |
| | |
Fixes #64746
|
| |\ \
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
into 'master'
Remove project param from `#show_label_issuables_link?`
Closes #62217
See merge request gitlab-org/gitlab-ce!31038
|
| | | |
| | |
| | |
| | | |
The project param is unnecessary here
|
| | | |
| | |
| | |
| | | |
Signed-off-by: Istvan szalai <istvan.szalai@savoirfairelinux.com>
|
| |/ /
| |
| |
| |
| |
| | |
Introduces new `feature_entry` helper for dashboard.
This helper reduces code duplication when listing available features
and relevant links to configuration sections
|
