793765 - ValidateOptions request: 3rd "strategy" argument for handling unknown keys

Status: RESOLVED INCOMPLETE

(Add-on SDK Graveyard :: General, defect, P3)

Description

[Gregg Lind (Fx Strategy and Insights - Shield - Heartbeat )]

Use case:

* validate the keys I care about
* ignore other keys (which become essentially **kwargs)

Options for 'strategy' might be:

* strict | fail  -> current
* allow | ignore ->  unknown / unvalidated keys are fine, and pass through
* (a function) ->  run on all unknown keys?


(previous discussion around enhancements:  https://bugzilla.mozilla.org/show_bug.cgi?id=569122)

Comment 1

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

Gregg, did you still need this?

Comment 2

[Gregg Lind (Fx Strategy and Insights - Shield - Heartbeat )]

Debatable.  I built around this... 

Having ValidateOptions around is nearly as powerful JSONSchema.  My own solution here is gross.  I would rather see some power expansion in the core (options:  omit, allow, 'function'). 

======
let validateOptionsLiberal = exports.validateOptionsLiberal = function(options,requirements){
	let out = validateOptions(options,requirements);
	for (let k in options) {
		if (! (k in requirements)){
			out[k] = options[k] // unvalidated
		}
	}
	return out
}
=========

Comment 3

[Firefox Bug Husbandry Bot]

https://bugzilla.mozilla.org/show_bug.cgi?id=1399562