Bug: tools.checkRangeQuery does not consider edge case of update.messages.length === 0

[titanism]

As you can see from code snippet below , if the case is that update.messages.length is 0, then an invalid query object will be returned. This namely happens for queryAll cases (e.g. 1:*) for COPY.

wildduck/lib/handlers/on-copy.js

Lines 66 to 73 in c9188b3

	let cursor = await db.database
	.collection('messages')
	.find({
	mailbox: mailboxData._id,
	uid: tools.checkRangeQuery(update.messages)
	}) // no projection as we need to copy the entire message
	.sort({ uid: 1 })
	.maxTimeMS(consts.DB_MAX_TIME_MESSAGES);

> const tools = require('wildduck/lib/tools')
undefined
> tools.checkRangeQuery([], false)
{ '$gte': undefined, '$lte': undefined }
> tools.checkRangeQuery([], true)
{ '$not': { '$gte': undefined, '$lte': undefined } }

[titanism]

The fix may also be needed here

wildduck/lib/message-handler.js

Line 845 in c9188b3

uid: options.messageQuery ? options.messageQuery : tools.checkRangeQuery(options.messages)