ottobackwards commented on issue #3331: NIFI-6076 syslog5424 should support 
missing MSG
URL: https://github.com/apache/nifi/pull/3331#issuecomment-467221426
 
 
   @bbende thanks for checking.  The issue I fixed is with the grammar.  It 
'fails' in a different way depending on the structure of the message.  In the 
test I added, it _logically_ fails.  The second structured data ends up _being_ 
the message body, but the event is valid.
   That is because there are two structured messages.
   If there where one structured message, then it would fail parse with a 
missing space.
   
   I need to add better tests.  Explicit tests.  I will do so and update.  Hope 
that makes sense, if you look at what I fixed in the grammar it should.
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to