teragrep / rlp_03

Java RELP Server library
GNU Affero General Public License v3.0
1 stars 5 forks source link

Split response status id away from payload #219

Open StrongestNumber9 opened 4 months ago

StrongestNumber9 commented 4 months ago

Description

RelpFrameAccess{txn=3, command=rsp, payloadLength=45, payload=500 EXCEPTION WHILE PROCESSING SYSLOG PAYLOAD, endOfTransfer= }

->

RelpFrameAccess{txn=3, command=rsp, payloadLength=45, status=500, payload=EXCEPTION WHILE PROCESSING SYSLOG PAYLOAD, endOfTransfer= }

Use case or motivation behind the feature request

Easier to handle, no need to do .startsWith("500") checks or anything

Related issues

Additional context

StrongestNumber9 commented 4 months ago

Alternatively leave it as-is and add secondary state attribute?

kortemik commented 1 month ago

payload is a payload, and basically has nothing to do with relp protocol itself it is part of the syslog transport on top of relp. i guess it could be made pluggable so we could have another module to implement the actual syslog transport