Skip to content

Remove clickhouse_connection_string ignore rule

Max Woolf requested to merge 409355-remove-clickhouse-connection-string into master

What does this MR do and why?

⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠
⚠ Do not merge until the 16.0 release has been finalized and cut. This MR cannot be released in 16.0. ⚠
⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠

  • Removes the ignore rule on application_settings.clickhouse_connection_string. This has been superceeded by an encrypted column to store this potentially sensitive field.
  • The column was successfully dropped in %16.0.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #409355 (closed)

Merge request reports