[
https://issues.apache.org/jira/browse/ARTEMIS-5890?focusedWorklogId=1003510&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-1003510
]
ASF GitHub Bot logged work on ARTEMIS-5890:
-------------------------------------------
Author: ASF GitHub Bot
Created on: 04/Feb/26 15:50
Start Date: 04/Feb/26 15:50
Worklog Time Spent: 10m
Work Description: clebertsuconic commented on code in PR #6218:
URL: https://github.com/apache/artemis/pull/6218#discussion_r2764702625
##########
artemis-protocols/artemis-amqp-protocol/src/main/java/org/apache/activemq/artemis/protocol/amqp/proton/AMQPLargeMessageWriter.java:
##########
@@ -173,6 +173,11 @@ private void tryDelivering() {
logger.trace("AMQP Large Message Writer was closed before queued
write attempt was executed");
return;
}
+ if (protonSender.getSession().getConnection().getRemoteState() !=
EndpointState.ACTIVE ||
protonSender.getSession().getConnection().getLocalState() !=
EndpointState.ACTIVE) {
Review Comment:
BTW: The thread being used here is the netty thread, as this is started with
connection.runLater calls. so this is in the same thread.
Issue Time Tracking
-------------------
Worklog Id: (was: 1003510)
Time Spent: 1h (was: 50m)
> Closing a connection while large message is being delivered could show
> exceptions in the log
> --------------------------------------------------------------------------------------------
>
> Key: ARTEMIS-5890
> URL: https://issues.apache.org/jira/browse/ARTEMIS-5890
> Project: Artemis
> Issue Type: Bug
> Reporter: Clebert Suconic
> Assignee: Clebert Suconic
> Priority: Major
> Labels: pull-request-available
> Fix For: 2.52.0
>
> Time Spent: 1h
> Remaining Estimate: 0h
>
> If you close a connection while the server still delivering large messages,
> you could see exceptions in the logs (IllegalArgumentException).
> The delivery thread should check the state of the connection before issuing
> more deliveries.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
