Commit 267f4299 authored by Fabio Pitino's avatar Fabio Pitino

Merge branch 'ci-remove-ci_builds_tokens_required_encryption-ff' into 'master'

Require encryption for CI job tokens

See merge request gitlab-org/gitlab!65166
parents bfc8bcf9 e7041029
...@@ -193,8 +193,7 @@ module Ci ...@@ -193,8 +193,7 @@ module Ci
acts_as_taggable acts_as_taggable
add_authentication_token_field :token, add_authentication_token_field :token, encrypted: :required
encrypted: -> { Gitlab::Ci::Features.require_builds_token_encryption? ? :required : :optional }
before_save :ensure_token before_save :ensure_token
before_destroy { unscoped_project } before_destroy { unscoped_project }
......
---
name: ci_builds_tokens_required_encryption
introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/63874
rollout_issue_url: https://gitlab.com/gitlab-org/gitlab/-/issues/333566
milestone: '14.0'
type: development
group: group::pipeline execution
default_enabled: false
...@@ -29,10 +29,6 @@ module Gitlab ...@@ -29,10 +29,6 @@ module Gitlab
def self.gldropdown_tags_enabled? def self.gldropdown_tags_enabled?
::Feature.enabled?(:gldropdown_tags, default_enabled: :yaml) ::Feature.enabled?(:gldropdown_tags, default_enabled: :yaml)
end end
def self.require_builds_token_encryption?
Feature.enabled?(:ci_builds_tokens_required_encryption, default_enabled: :yaml)
end
end end
end end
end end
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