Improper handling of container restrictions in creation of sessions
-------------------------------------------------------------------
Key: AMQ-1618
URL: https://issues.apache.org/activemq/browse/AMQ-1618
Project: ActiveMQ
Issue Type: Bug
Components: Connector, Geronimo Integration
Affects Versions: 5.0.0, 4.1.1
Environment: All
Reporter: Manu T George
Fix For: 4.1.2, 5.1.0, 5.2.0
In the getServerSession() method of
org.apache.activemq.ra.ServerSessionPoolImpl there is an issue with the code
block given below
ServerSessionImpl ss = createServerSessionImpl();
// We may not be able to create a session due to the container
// restricting us.
if (ss == null) {
if (idleSessions.size() == 0) {
throw new JMSException("Endpoint factory did not allows to
any endpoints.");
}
return getExistingServerSession();
}
If the container restricts the creation of a session then messages should be
queued to an existing session. So the check should be
if (activeSessions.size() == 0) {
throw new JMSException("Endpoint factory did not allows to
any endpoints.");
}
instead of the one above
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
