activemq-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Manu T George (JIRA)" <j...@apache.org>
Subject [jira] Created: (AMQ-1618) Improper handling of container restrictions in creation of sessions
Date Wed, 12 Mar 2008 10:46:35 GMT
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.


Mime
View raw message