Inconsistency between fbs and worker

[ybybwdwd]

In request.fbs, the Request.handler_id is required,

table Request {
    id: uint32;
    method: Method;
    handler_id: string (required);
    body: Body;
}

but in worker ChannelRequest::ChannelRequest():110 checks if the field exists

// Handler ID is optional.
if (flatbuffers::IsFieldPresent(this->data, FBS::Request::Request::VT_HANDLERID))
{
    this->handlerId = this->data->handlerId()->str();
}

[ibc]

@jmillan I think we can safely remove that check in Worker.cpp, right?

[jmillan]

Yep, that check can be removed.

EDIT: I'm quite sure, but needs to be checked.

[ibc]

Same applies to ChannelNotification.cpp. On it.