Issue Details (XML | Word | Printable)

Key: AMQ-1365
Type: Bug Bug
Status: Resolved Resolved
Resolution: Fixed
Priority: Major Major
Assignee: Unassigned
Reporter: William Lam
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
ActiveMQ

Username and password field got swapped when calling PooledConnection.createConnection

Created: 17/Aug/07 01:52 PM   Updated: 04/Jan/08 02:14 PM
Return to search
Component/s: JMS client
Affects Version/s: 4.1.1
Fix Version/s: 4.1.2, 5.1.0

Time Tracking:
Not Specified

Environment: Windows XP


 Description  « Hide
When using org.apache.activemq.pool.PooledConnection createConnection(username, password), the username and password fields are swapped.

I think the problem could be the following, based in 4.1.1 source code,

PooledConnection.java line 77

public synchronized Connection createConnection(String userName, String password) throws JMSException {
ConnectionKey key = new ConnectionKey(userName, password);

ConnectionKey.java line 30

public ConnectionKey(String password, String userName) {
this.password = password;
this.userName = userName;



 All   Comments   Work Log   Change History   Subversion Commits   FishEye   Crucible      Sort Order: Ascending order - Click to sort in descending order
Rob Davies made changes - 19/Aug/07 10:37 PM
Field Original Value New Value
Fix Version/s 5.2.0 [ 11841 ]
568868 by  Jonas Lim (1 file)
23/Aug/07 12:46 AM (26 months, 22 days ago)
568869 by  Jonas Lim (1 file)
23/Aug/07 12:48 AM (26 months, 22 days ago)
Repository Revision Date User Message
AMQ #568869 Thu Aug 23 00:48:37 PDT 2007 jlim fix for https://issues.apache.org/activemq/browse/AMQ-1365
Files Changed
MODIFY /activemq/trunk/activemq-core/src/main/java/org/apache/activemq/pool/ConnectionKey.java

Jonas Lim added a comment - 23/Aug/07 12:49 AM
Thanks!

Updated the 4.1 branch and trunk to address this issue :

4.1 branch : rev 568868
trunk : rev 568869


Jonas Lim made changes - 23/Aug/07 12:49 AM
Status Open [ 1 ] Resolved [ 5 ]
Fix Version/s 4.1.2 [ 11801 ]
Resolution Fixed [ 1 ]
Hiram Chirino made changes - 04/Jan/08 02:14 PM
Fix Version/s 5.2.0 [ 11841 ]
Fix Version/s 5.1.0 [ 11802 ]
Create crucible review for all 2 changesets in