Rename "white list"

Can we rename the term “white list” as found in default controller scaffold to something a little more inclusive?

2 Likes

It’s better to change to the darklist and sunnylist :slight_smile:

It seems this was partially completed in 2018 (Replace use of whitelist with allowlist and blacklist with denylist · Issue #33677 · rails/rails · GitHub), but that was for the term “whitelist” without a space. A quick search on the codebase just shows 3 remaining, but the most glaring is on a scaffolded controller. Search · "white list" · GitHub

@Dmitry_Polushkin Thank you for the suggestion but I don’t think this brings anything to the discussion.

@nphaskins those are all fixed now.

2 Likes

The term “whitelist” was fixed, but the term “white list” (without a space) still exists. I linked to the source in the codebase.

“# Only allow a trusted parameter “white list” through.”

https://github.com/rails/rails/commit/2c5c452d8cf3ee8e042aa4c0187b32a88d0da025

You are fast @rafaelfranca ! Thank you!! :heart_eyes: :smiling_face_with_three_hearts:

3 Likes