Parameters set in the clientRequestProperties are not sanitized to "dynamic" when needed

[johnib]

Hey,

Could not find the place where the parameters are converted to their kusto types, as described here.

For example, if I specify a date parameter:

clientRequestProperties.setParameter('startTime', new Date().toISOString());

Then something in your package will wrap the value with datetime(<value>).
However, if I provide an array of values:

clientRequestProperties.setParameter('ids', [1,2,3]);

The package will not sanitize it to dynamic(<value>).

Tried looking for the actual code doing this sanitization, however I couldn't find it.

azure-kusto-node/azure-kusto-data/source/clientRequestProperties.ts

Lines 64 to 80 in 6f77491

	toJson() {
	const json: { Options?: { [option: string]: any }, Parameters?: { [option: string]: any } } = {};
	if (Object.keys(this._options).length !== 0) {
	json.Options = this._options;
	if (json.Options.servertimeout) {
	json.Options.servertimeout = this._msToTimespan(json.Options.servertimeout);
	}
	}
	if (Object.keys(this._parameters).length !== 0) {
	json.Parameters = this._parameters;
	}
	return Object.keys(json).length !== 0 ? json : null;
	}

[yihezkel]

Hi @johnib ,

We will add this to our backlog.

As a workaround, please explicitly add the dynamic wrapper as needed.

CC @yogilad