RpcProxyFactory always produced console error log

[martin-fleck-at]

Feature Description:

There is some code in the RpcProxyFactory that always logs an error even though the error is re-thrown and could by handled by a client, thus spamming the log unnecessarily:

theia/packages/core/src/common/messaging/proxy-factory.ts

Lines 163 to 180 in d0a09a9

	protected async onRequest(method: string, ...args: any[]): Promise<any> {
	try {
	if (this.target) {
	return await this.target[method](...args);
	} else {
	throw new Error(`no target was set to handle ${method}`);
	}
	} catch (error) {
	const e = this.serializeError(error);
	if (e instanceof ResponseError) {
	throw e;
	}
	const reason = e.message || '';
	const stack = e.stack || '';
	console.error(`Request ${method} failed with error: ${reason}`, stack);
	throw e;
	}
	}

I suggest to remove that code or at least only provide it through a logger that can be provided.