Skip to content

Commit

Permalink
Add deprecation levels for other config deprecations
Browse files Browse the repository at this point in the history
Note: I skipped `xpack.security.enabled`, `xpack.spaces.enabled`, and
the legacy audit logger because those are being removed in 8.0 / they
have their own 7.x-specific PRs to make those changes.
  • Loading branch information
jportner committed Oct 18, 2021
1 parent 2c66a36 commit b409027
Showing 1 changed file with 11 additions and 10 deletions.
21 changes: 11 additions & 10 deletions x-pack/plugins/security/server/config_deprecations.ts
Original file line number Diff line number Diff line change
Expand Up @@ -13,22 +13,23 @@ export const securityConfigDeprecationProvider: ConfigDeprecationProvider = ({
renameFromRoot,
unused,
}) => [
rename('sessionTimeout', 'session.idleTimeout'),
rename('authProviders', 'authc.providers'),
rename('sessionTimeout', 'session.idleTimeout', { level: 'warning' }),
rename('authProviders', 'authc.providers', { level: 'warning' }),

rename('audit.appender.kind', 'audit.appender.type'),
rename('audit.appender.layout.kind', 'audit.appender.layout.type'),
rename('audit.appender.policy.kind', 'audit.appender.policy.type'),
rename('audit.appender.strategy.kind', 'audit.appender.strategy.type'),
rename('audit.appender.path', 'audit.appender.fileName'),
rename('audit.appender.kind', 'audit.appender.type', { level: 'warning' }),
rename('audit.appender.layout.kind', 'audit.appender.layout.type', { level: 'warning' }),
rename('audit.appender.policy.kind', 'audit.appender.policy.type', { level: 'warning' }),
rename('audit.appender.strategy.kind', 'audit.appender.strategy.type', { level: 'warning' }),
rename('audit.appender.path', 'audit.appender.fileName', { level: 'warning' }),

renameFromRoot(
'security.showInsecureClusterWarning',
'xpack.security.showInsecureClusterWarning'
'xpack.security.showInsecureClusterWarning',
{ level: 'warning' }
),

unused('authorization.legacyFallback.enabled'),
unused('authc.saml.maxRedirectURLSize'),
unused('authorization.legacyFallback.enabled', { level: 'warning' }),
unused('authc.saml.maxRedirectURLSize', { level: 'warning' }),
// Deprecation warning for the legacy audit logger.
(settings, fromPath, addDeprecation, { branch }) => {
const auditLoggingEnabled = settings?.xpack?.security?.audit?.enabled ?? false;
Expand Down

0 comments on commit b409027

Please sign in to comment.