brusdev commented on issue #2694: ARTEMIS-2371 Message with huge header shuts 
broker down
URL: https://github.com/apache/activemq-artemis/pull/2694#issuecomment-500116229
 
 
   @clebertsuconic I changed the exception with a more specific one and I 
checked that the exception was already logged by 
ProtonServerReceiverContext.actualDelivery.
   @franz1981 I added a check to avoid to to allocate resources for a large 
message if it can't be stored, this check compares large message encoded size 
(without body) with the max record size (for JournalStorageManager is the 
minimum between file size and buffer size).
   @michaelandrepearce I need to estimate header size at protocol layer to make 
a check and i think that this evaluations is too heavy.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to 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