Remove the term whitelist* from the module

Created on 13 August 2025, 1 day ago

Problem/Motivation

Blacklist and whitelist have been added to the list of flagwords
I would suggest these should be replaced with something from the list suggested by google:
https://developers.google.com/style/word-list#blacklist

Steps to reproduce

Search the code for white list

Proposed resolution

Replace with the term allowlist

📌 Task
Status

Active

Version

2.0

Component

Code

Created by

🇬🇧United Kingdom the_g_bomb

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

  • Issue created by @the_g_bomb
  • @the_g_bomb opened merge request.
  • 🇳🇿New Zealand jweowu

    You possibly need to make sure that:

    1. The value being written to the database is only the value which was in the database to start with (as opposed to some code-based override of that value that you're obtaining at run-time).

    2. The code still checks for the original config name (which again, might be getting set in a code override), and merges it in appropriately if a value exists.

    Take Provide hook_seckit_options_alter() D8 Needs review into consideration, at minimum, but there might be other possibilities. I commented on some in https://www.drupal.org/project/seckit/issues/2844205#comment-13495176 Provide hook_seckit_options_alter() D8 Needs review but that was quite a long time ago, and I don't know what he current state of affairs is.

Production build 0.71.5 2024