| Commit message (Collapse) | Author | Age | Files | Lines |
| | |
|
| | |
|
| |\
| |
| |
| |
| |
| |
| |
| |
| | |
Enable the Rubocop DeprecatedClassMethods cop
This reports uses of `File.exists?` and `Dir.exists?`, which were both
deprecated in Ruby and will eventually be removed in favor of `.exist?`.
Also fixes all existing uses of the deprecated methods.
See merge request !4087
|
| | |
| |
| |
| |
| |
| | |
This reports uses of `File.exists?` and `Dir.exists?`, which were both
deprecated in Ruby and will eventually be removed in favor of
`.exist?`. Also fixes all existing uses of the deprecated methods.
|
| |\ \
| | |
| | |
| | |
| | | |
# Conflicts:
# db/schema.rb
|
| | | |
| | |
| | |
| | | |
health_check feature.
|
| | | |
| | |
| | |
| | | |
Also added a health check page to the admin section for resetting the token.
|
| | | | |
|
| | | | |
|
| |\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Update build log incrementally
Proof of concept implementation of incremental sending of build log to browser.
cc @jschatz1 @vsizov @grzesiek @tmaczukin
See merge request !3737
|
| | |\ \ \
| | |/ / |
|
| | | |/
| |/|
| | |
| | |
| | | |
We send a state of ansi2html to client, client needs to send this state back.
The state describes the configuration of generator and position within trace.
|
| |\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Add snippet tab under user profile
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/15572
See merge request !4001
|
| | |\ \ \
| | | | |
| | | | |
| | | | | |
issue_15572_snippets_tab_under_user_profile
|
| | | | | | |
|
| | | | | | |
|
| | | | | | |
|
| | | | | | |
|
| | |\ \ \ \
| | | |/ /
| | |/| |
| | | | | |
issue_15572_snippets_tab_under_user_profile
|
| | | | | | |
|
| | |_|/ /
|/| | |
| | | |
| | | | |
Signed-off-by: Rémy Coutable <remy@rymai.me>
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
- Fix naming of API::CommitStatuses
- Ensure we use require_dependency instead of require
- Ensure the namespace is right in lib/api/api.rb, otherwise, we
might require Grape::API::Helpers which defines the `#params` method.
This is to avoid requiring a file multiple times and getting an "Already
initialized constant" error.
Signed-off-by: Rémy Coutable <remy@rymai.me>
|
| | |_|/
|/| |
| | |
| | |
| | |
| | | |
Attempts to address #13521, #14825
Signed-off-by: Rémy Coutable <remy@rymai.me>
|
| |\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Remove `wall_enabled` field from Project
See merge request !4089
|
| | | |/
| |/| |
|
| |\ \ \
| |/ /
|/| |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Pass trusted_proxies to action_dispatch as IPAddrs instead of strings
Without this setting your own trusted_proxies does not work.
Fixes an issue introduce in: https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/3524
Fixes: https://gitlab.com/gitlab-org/gitlab-ce/issues/17004
See merge request !3970
|
| | | |
| | |
| | |
| | | |
Without this setting your own trusted_proxies does not work.
|
| | | | |
|
| |\ \ \
| | | |
| | | |
| | | |
| | | | |
Instrument methods used in email diffs
See merge request !4038
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Make all of the nested constant instrumentation for core app code work
the same way, add mailer instrumentation, and add instrumentation to the
premailer gem.
|
| |/ / /
| | |
| | |
| | |
| | |
| | | |
This reverts commit 1cc614f2bdd30b4fce35ee9e680f9272b9012978.
It was causing the ActiveJob integration to fail, so unfortunately we'll have to add the gem again.
|
| |\ \ \ |
|
| | | |/
| |/| |
|
| | | | |
|
| |\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Add more parameters to the filter_parameters config.
Adds Sentry DSN, Webhooks, Deploy Keys, etc.
Alphabetized the parameters and included line breaks between each parameter. Easier to merge into EE if there are any differences.
This also seems to be the more popular syntax for adding new parameters, from what I can find.
In the future we may want to [increase the specificity of some of these](http://blog.bigbinary.com/2016/03/07/parameter-filtering-enhacement-rails-5.html) once Rails 5 is out. e.g. instead of `:hook`, `"hook.url"` would be just as effective without removing potentially useful information from the logs.
cc: @stanhu
See merge request !3971
|
| | | |/
| |/|
| | |
| | |
| | |
| | |
| | |
| | | |
Adds Sentry DSN, Webhooks, Deploy Keys, etc.
Alphabetized the parameters and included line breaks between each parameter. Easier to merge into EE if there are any differences.
This also seems to be the more popular syntax for adding new parameters, from what I can find.
|
| |\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Prevent Rails filtered parameters from leaking to Sentry.
As described in their Docs: https://docs.getsentry.com/on-premise/clients/ruby/integrations/rails/
See merge request !3974
|
| | |/ /
| | |
| | |
| | | |
As described in their Docs: https://docs.getsentry.com/on-premise/clients/ruby/integrations/rails/
|
| |/ /
| |
| |
| |
| |
| |
| | |
This button depends on Issue#can_be_worked_on? which in turn depends on
Issue#related_branches. By rendering this button asynchronously we can
finally remove all usages of Issue#related_branches and
Issue#referenced_merge_requests from the issues "show" page.
|
| |\ \
| | |
| | |
| | |
| | |
| | |
| | | |
Disable Rack Attack if admin disables it in config file
Grack::Auth already checks this variable. These holdouts were not disabled, leading to confusion when debugging a customer issue.
See merge request !3918
|
| | | |
| | |
| | |
| | |
| | | |
Grack::Auth already checks this variable. These holdouts were not
disabled, leading to confusion when debugging a customer issue.
|
| |\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Send 'admin emails' weekly, not daily
Daily seems to be to spammy, so let's default to weekly instead.
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/15520
See merge request !3923
|
| | | |/
| |/|
| | |
| | | |
Daily seems to be to spammy, so let's default to weekly instead.
|
| |\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Backport GitHub Enterprise import support from EE
These changes were pulled from GitLab EE to support configuring
an alternative API URL than the default https://api.github.com.
In addition, the `verify_ssl` flag allows users to disable SSL cert
checking.
One modification: add a default `args` option if it does not exist
to avoid breaking existing configurations.
See merge request !3892
|
| | | | | |
|
| | | |/
| |/|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
These changes were pulled from GitLab EE to support configuring
an alternative API URL than the default https://api.github.com.
In addition, the `verify_ssl` flag allows users to disable SSL cert
checking.
One modification: add a default `args` option if it does not exist
to avoid breaking existing configurations.
|
| |\ \ \
| |/ /
|/| |
| | |
| | |
| | |
| | |
| | |
| | | |
Add letter_opener_web gem
Sent emails in the development environment will no longer automatically
be opened in the browser. If a sent email needs to be viewed, it can be
found at the `/rails/letter_opener` route.
See merge request !3893
|
| | |/
| |
| |
| |
| |
| | |
Sent emails in the development environment will no longer automatically
be opened in the browser. If a sent email needs to be viewed, it can be
found at the `/rails/letter_opener` route.
|
| |/
|
|
|
|
| |
The extra gem isn’t necessary anymore since Rails 4.2 has ActiveJob integration.
Resolves #15575.
|
| |
|
|
| |
Closes #4228
|