I'm perceiving a couple of issues with how the list of fields to sanitize is handled.
- The language is not clear. When I say
config.sanitize_fields = %w(passwd poop monkeybutt), I intuit that those are the only fields that will be sanitized. In reality, those are added to an existing list of default fields.
- There's no option to change/replace the default list. The default list is helpful, but I may have a legitimate need to exclude one or more of the default fields.
Ideas:
# replacing the entire list
config.sanitize_fields = %w(passwd poop monkeybutt)
# adding fields to the list
config.sanitize_fields |= %w(passwd poop monkeybutt)
# or
config.sanitize_additional_fields = %w(passwd poop monkeybutt)
Thoughts?
I'm perceiving a couple of issues with how the list of fields to sanitize is handled.
config.sanitize_fields = %w(passwd poop monkeybutt), I intuit that those are the only fields that will be sanitized. In reality, those are added to an existing list of default fields.Ideas:
Thoughts?