diff options
author | GitLab Bot <gitlab-bot@gitlab.com> | 2020-04-07 09:09:13 +0000 |
---|---|---|
committer | GitLab Bot <gitlab-bot@gitlab.com> | 2020-04-07 09:09:13 +0000 |
commit | 41cb558299b483b44b45351730ee4c0e9fe4ca2c (patch) | |
tree | 00688a9e40021c66195ad826ceddd9c19385cdcf /lib | |
parent | 3d064c737e8448880e6180aeddc59000a01aa6a8 (diff) | |
download | gitlab-ce-41cb558299b483b44b45351730ee4c0e9fe4ca2c.tar.gz |
Add latest changes from gitlab-org/gitlab@master
Diffstat (limited to 'lib')
-rw-r--r-- | lib/api/internal/base.rb | 2 | ||||
-rw-r--r-- | lib/gitlab/ci/templates/Managed-Cluster-Applications.gitlab-ci.yml | 2 | ||||
-rw-r--r-- | lib/gitlab/sidekiq_logging/json_formatter.rb | 5 | ||||
-rw-r--r-- | lib/gitlab/sidekiq_logging/logs_jobs.rb | 2 |
4 files changed, 8 insertions, 3 deletions
diff --git a/lib/api/internal/base.rb b/lib/api/internal/base.rb index 927639599d2..1e528d7a691 100644 --- a/lib/api/internal/base.rb +++ b/lib/api/internal/base.rb @@ -108,7 +108,7 @@ module API # check_ip - optional, only in EE version, may limit access to # group resources based on its IP restrictions post "/allowed" do - if repo_type.snippet? && Feature.disabled?(:version_snippets, actor.user) + if repo_type.snippet? && params[:protocol] != 'web' && Feature.disabled?(:version_snippets, actor.user) break response_with_status(code: 401, success: false, message: 'Snippet git access is disabled.') end diff --git a/lib/gitlab/ci/templates/Managed-Cluster-Applications.gitlab-ci.yml b/lib/gitlab/ci/templates/Managed-Cluster-Applications.gitlab-ci.yml index 4ef6a4d3bef..713b11c4d8f 100644 --- a/lib/gitlab/ci/templates/Managed-Cluster-Applications.gitlab-ci.yml +++ b/lib/gitlab/ci/templates/Managed-Cluster-Applications.gitlab-ci.yml @@ -1,6 +1,6 @@ apply: stage: deploy - image: "registry.gitlab.com/gitlab-org/cluster-integration/cluster-applications:v0.12.0" + image: "registry.gitlab.com/gitlab-org/cluster-integration/cluster-applications:v0.13.1" environment: name: production variables: diff --git a/lib/gitlab/sidekiq_logging/json_formatter.rb b/lib/gitlab/sidekiq_logging/json_formatter.rb index c20e929ae36..45c6842c59b 100644 --- a/lib/gitlab/sidekiq_logging/json_formatter.rb +++ b/lib/gitlab/sidekiq_logging/json_formatter.rb @@ -19,6 +19,7 @@ module Gitlab output[:message] = data when Hash convert_to_iso8601!(data) + stringify_args!(data) output.merge!(data) end @@ -39,6 +40,10 @@ module Gitlab Time.at(timestamp).utc.iso8601(3) end + + def stringify_args!(payload) + payload['args'] = Gitlab::Utils::LogLimitedArray.log_limited_array(payload['args'].map(&:to_s)) if payload['args'] + end end end end diff --git a/lib/gitlab/sidekiq_logging/logs_jobs.rb b/lib/gitlab/sidekiq_logging/logs_jobs.rb index 55d711c54ae..326dfdae661 100644 --- a/lib/gitlab/sidekiq_logging/logs_jobs.rb +++ b/lib/gitlab/sidekiq_logging/logs_jobs.rb @@ -7,6 +7,7 @@ module Gitlab "#{payload['class']} JID-#{payload['jid']}" end + # NOTE: Arguments are truncated/stringified in sidekiq_logging/json_formatter.rb def parse_job(job) # Error information from the previous try is in the payload for # displaying in the Sidekiq UI, but is very confusing in logs! @@ -16,7 +17,6 @@ module Gitlab job['pid'] = ::Process.pid job.delete('args') unless ENV['SIDEKIQ_LOG_ARGUMENTS'] - job['args'] = Gitlab::Utils::LogLimitedArray.log_limited_array(job['args'].map(&:to_s)) if job['args'] job end |