[
https://issues.apache.org/jira/browse/AMQ-9478?focusedWorklogId=947383&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-947383
]
ASF GitHub Bot logged work on AMQ-9478:
---------------------------------------
Author: ASF GitHub Bot
Created on: 09/Dec/24 18:11
Start Date: 09/Dec/24 18:11
Worklog Time Spent: 10m
Work Description: pa-deasy commented on code in PR #1360:
URL: https://github.com/apache/activemq/pull/1360#discussion_r1876462030
##########
activemq-unit-tests/src/test/java/org/apache/activemq/ActiveMQSslConnectionFactoryTest.java:
##########
@@ -72,6 +81,34 @@ public void testCreateTcpConnectionUsingKnownPort() throws
Exception {
brokerStop();
}
+ public void testCreateTcpConnectionWithSocketParameters() throws Exception
{
+ // Control case: check that the factory can create an ordinary
(non-ssl) connection.
+ String tcpUri =
"tcp://localhost:61610?socket.OOBInline=true&socket.keepAlive=true&tcpNoDelay=true";
+ broker = createBroker(tcpUri);
+
+ // This should create the connection.
+ ActiveMQSslConnectionFactory cf = getFactory(tcpUri);
+ connection = (ActiveMQConnection)cf.createConnection();
+ assertNotNull(connection);
+
+ Transport transport = connection.getTransport();
Review Comment:
Good suggestion. I initially thought about keeping the logic separate as one
test gets a `Socket` and the other gets an `SSLSocket`. A simple check to
determine whether or not to get the superclass makes the helper method fit both
tests though.
Issue Time Tracking
-------------------
Worklog Id: (was: 947383)
Time Spent: 1.5h (was: 1h 20m)
> Add unit test for modernized IntrospectionSupport
> -------------------------------------------------
>
> Key: AMQ-9478
> URL: https://issues.apache.org/jira/browse/AMQ-9478
> Project: ActiveMQ Classic
> Issue Type: Task
> Reporter: Matt Pavlovich
> Assignee: Jean-Baptiste Onofré
> Priority: Minor
> Fix For: 6.2.0
>
> Time Spent: 1.5h
> Remaining Estimate: 0h
>
> Update unit test to verify transport connector and network connector uri
> handling for
> IntrospectionSupport to deal with TcpSocket properties
> IntrospectionSupport to deal with SSLSocket properties
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
For further information, visit: https://activemq.apache.org/contact
