Commit 32062a91 authored by Robert Speicher's avatar Robert Speicher

Merge branch 'sh-allow-key-id-in-params' into 'master'

Filter any parameters ending with "key" in logs

See merge request gitlab-org/gitlab-ce!21688
parents a2be6536 5c79c68e
---
title: Filter any parameters ending with "key" in logs
merge_request: 21688
author:
type: changed
...@@ -85,6 +85,7 @@ module Gitlab ...@@ -85,6 +85,7 @@ module Gitlab
# - Any parameter ending with `token` # - Any parameter ending with `token`
# - Any parameter containing `password` # - Any parameter containing `password`
# - Any parameter containing `secret` # - Any parameter containing `secret`
# - Any parameter ending with `key`
# - Two-factor tokens (:otp_attempt) # - Two-factor tokens (:otp_attempt)
# - Repo/Project Import URLs (:import_url) # - Repo/Project Import URLs (:import_url)
# - Build traces (:trace) # - Build traces (:trace)
...@@ -92,15 +93,13 @@ module Gitlab ...@@ -92,15 +93,13 @@ module Gitlab
# - GitLab Pages SSL cert/key info (:certificate, :encrypted_key) # - GitLab Pages SSL cert/key info (:certificate, :encrypted_key)
# - Webhook URLs (:hook) # - Webhook URLs (:hook)
# - Sentry DSN (:sentry_dsn) # - Sentry DSN (:sentry_dsn)
# - Deploy keys (:key)
# - File content from Web Editor (:content) # - File content from Web Editor (:content)
config.filter_parameters += [/token$/, /password/, /secret/] config.filter_parameters += [/token$/, /password/, /secret/, /key$/]
config.filter_parameters += %i( config.filter_parameters += %i(
certificate certificate
encrypted_key encrypted_key
hook hook
import_url import_url
key
otp_attempt otp_attempt
sentry_dsn sentry_dsn
trace trace
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment