If the priority range of a proto / flatbuffer connection is not in the valid range (100 - 199) we will send an error message to the client as well as in the console output

Signed-off-by: Paulchen-Panther <Paulchen-Panter@protonmail.com>
This commit is contained in:
Paulchen-Panther 2019-08-12 19:06:11 +02:00
parent 0f839f670e
commit a8a6113517
No known key found for this signature in database
GPG Key ID: 84E3B692456B6840
2 changed files with 5 additions and 3 deletions

View File

@ -123,7 +123,7 @@ void FlatBufferClient::handleRegisterCommand(const hyperionnet::Register *regReq
{ {
if (regReq->priority() < 100 || regReq->priority() >= 200) if (regReq->priority() < 100 || regReq->priority() >= 200)
{ {
// Warning(_log, "Register request from client %s contains invalid priority %d. Valid rage is between 100 and 199.", QSTRING_CSTR(_clientAddress), regReq->priority()); Error(_log, "Register request from client %s contains invalid priority %d. Valid priority for Flatbuffer connections is between 100 and 199.", QSTRING_CSTR(_clientAddress), regReq->priority());
sendErrorReply("The priority " + std::to_string(regReq->priority()) + " is not in the priority range between 100 and 199."); sendErrorReply("The priority " + std::to_string(regReq->priority()) + " is not in the priority range between 100 and 199.");
return; return;
} }

View File

@ -126,7 +126,8 @@ void ProtoClientConnection::handleColorCommand(const proto::ColorRequest &messag
if (priority < 100 || priority >= 200) if (priority < 100 || priority >= 200)
{ {
sendErrorReply("The priority " + std::to_string(priority) + " is not in the priority range between 100 and 199."); sendErrorReply("The priority " + std::to_string(priority) + " is not in the valid priority range between 100 and 199.");
Error(_log, "The priority %d is not in the proto-connection range between 100 and 199.", priority);
return; return;
} }
@ -156,7 +157,8 @@ void ProtoClientConnection::handleImageCommand(const proto::ImageRequest &messag
if (priority < 100 || priority >= 200) if (priority < 100 || priority >= 200)
{ {
sendErrorReply("The priority " + std::to_string(priority) + " is not in the priority range between 100 and 199."); sendErrorReply("The priority " + std::to_string(priority) + " is not in the valid priority range between 100 and 199.");
Error(_log, "The priority %d is not in the proto-connection range between 100 and 199.", priority);
return; return;
} }