I think for root behavior it makes the most sense to just leave the object alone.. If my schema looks like Joi.string().strip() it wouldn't make much sense to return undefined in my mind. Thoughts?

@Marsup what do you think about allowing to pass an array of keys to be removed from an object as well? something like

Joi.object({
  id: Joi.string(),
  password: Joi.string(),
  private: Joi.string()
}).strip(['password', 'private'])

This would be useful in when clauses, like:

Joi.object({
  id: Joi.string(),
  password: Joi.string(),
  private: Joi.string()
}).when('some condition', { is: true, then: Joi.strip(['password, 'private']) });

Why should the behavior be different just because it's root ? It seems inconsistent.
The syntax with parameters is tempting but isn't it weird that one applies to self and the other to children ?

Yeah, it does seem weird, it was a thought that would make stripping multiple keys under the same condition possible without having to repeat the condition. I started trying to implement that behavior though, and it becomes problematic quickly so I think it's probably not worth it.

This thread has been automatically locked due to inactivity. Please open a new issue for related bugs or questions following the new issue template instructions.