[#DIRMINA-166] Common internal messages that can be shared by filters and transport implementations

MINA

Common internal messages that can be shared by filters and transport implementations

Created: 17/Jan/06 11:27 AM Updated: 22/Nov/08 06:55 PM

Return to search

Component/s:

None

Affects Version/s:

0.9.0

Fix Version/s:

None

Time Tracking:

Not Specified

Description

« Hide

There are some common messages that can be shared by filters and transport implementations like in the SSL filter for handling secured and unsecured messages. Sometimes having this in the target requires JDK 1.5 dependency even if these features are not used. Here's a thread of converstation between Trustin and I about this speciifcially dealing with the SSLFilter and the LDAP protocol provider:

(21:19:02) trustin: StartTLS needs events that notify when the current session is secured or unsecured.
(21:19:14) trustin: SSLFilter emits them.
(21:19:21) akarasulu: right but can't we check the name of the class
(21:19:27) trustin: LDAP provider translates them into ExtendedMessage.
(21:19:46) trustin: We can do that.
(21:20:24) akarasulu: like message.getClass().equals( "org.apache.mina.filter.SSLFilter.SSLFilterMessage" );
(21:20:34) akarasulu: is this way bad?
(21:20:51) trustin: I think it's bad. We'd better define some core messages in mina-core.
(21:21:06) trustin: in... org.apache.mina.message.SessionSecured
(21:21:17) trustin: SessionUnsecured something like that.
(21:21:37) akarasulu: yeah that's better you're right

All

Comments

Work Log

Change History

Subversion Commits

Sort Order:

Repository	Revision	Date	User	Message
ASF	#370033	Wed Jan 18 03:53:40 UTC 2006	akarasulu	added notice of disconnect and hooked in protocolError trigger, waiting on DIRMINA-166 to complete the other trigger for unavailable resulttype
				Files Changed
				MODIFY /directory/trunks/apacheds/protocols/ldap/src/main/java/org/apache/ldap/server/protocol/LdapProtocolProvider.java ADD /directory/trunks/common/ldap/src/main/java/org/apache/ldap/common/message/extended/NoticeOfDisconnect.java ADD /directory/trunks/common/ldap/src/main/java/org/apache/ldap/common/message/extended MODIFY /directory/trunks/common/ldap/src/main/java/org/apache/ldap/common/codec/TwixTransformer.java

[ Permlink | « Hide ]

Alex Karasulu added a comment - 18/Jan/06 12:27 PM

Might also be a good idea to create a ServiceUnbound message class to send to the IoHandler via the messageRecieved() handler method. This message can tell the handler that it must do something to active sessions before the service is unbound and all active sessions are automatically closed. This in combination with the mechanism introduced by Niklas might be useful for responding to unbinds.

I will use this to send a Notice of Disconnect to LDAP clients before shutting down all sessions.

[ Permlink | « Hide ]

Alex Karasulu added a comment - 18/Jan/06 12:39 PM

DIRMINA-166 will add a ServiceUnbound message that is to be delivered to he IoHandler via the messageRecieved() method. This will be used to iterate through the active sessions and send the NoD then close each session. We need to wait for this feauture before we can finish this one.

Alex Karasulu made changes - 18/Jan/06 12:39 PM

Field	Original Value	New Value
Link		This issue blocks DIREVE-323 [ DIREVE-323 ]

Trustin Lee made changes - 24/Jan/06 11:13 AM

Fix Version/s		0.9.2 [ 12310730 ]
Fix Version/s	0.9.1 [ 12310371 ]

Trustin Lee made changes - 20/Feb/06 11:25 PM

Fix Version/s	0.9.2 [ 12310730 ]
Fix Version/s		0.9.3 [ 12310816 ]

Trustin Lee made changes - 20/Feb/06 11:25 PM

Link

This issue blocks ~~DIRSERVER-281~~ [ ~~DIRSERVER-281~~ ]

[ Permlink | « Hide ]

Trustin Lee added a comment - 26/Feb/06 09:27 AM

This issue is not urgent, so I'll schedule this issue later again.

Trustin Lee made changes - 26/Feb/06 09:27 AM

Fix Version/s

0.9.3 [ 12310816 ]

[ Permlink | « Hide ]

Emmanuel Lecharny added a comment - 22/Nov/08 06:55 PM

Status for this issue ?