[#DIRSERVER-711] Propagate the errors to the client if we get a DecoderException (was DIRSERVER-634)

Directory ApacheDS

Propagate the errors to the client if we get a DecoderException (was DIRSERVER-634)

Created: 22/Aug/06 11:41 PM Updated: 30/Aug/06 02:59 PM

Return to search

Component/s:

None

Affects Version/s:

None

Fix Version/s:

None

Time Tracking:

Not Specified

Issue Links:

Duplicate

This issue duplicates:

~~DIRSERVER-309~~ Decoder does no give sufficient information about errors so the LDAP server can respond with correct result code.

This issue is duplicated by:

~~DIRSERVER-309~~ Decoder does no give sufficient information about errors so the LDAP server can respond with correct result code.

Resolution Date:

30/Aug/06 02:59 PM

Description

« Hide

In codec grammars, when we have a decoderException, we should generate the correct LdapMessage response so the client can get the correct error instead of a protocol error.

All

Comments

Work Log

Change History

Subversion Commits

Sort Order:

Repository	Revision	Date	User	Message
ASF	#436848	Fri Aug 25 16:39:48 UTC 2006	elecharny	Fixed the potentiol hang if a clien send an invalid DN. Now a correct Response is send, with an error in LdapResult. This will fix ~~DIRSERVER-711~~, at least partially
				Files Changed
				MODIFY /directory/branches/shared/0.9.5/ldap/src/main/java/org/apache/directory/shared/ldap/codec/add/AddRequestGrammar.java MODIFY /directory/branches/shared/0.9.5/ldap/src/main/java/org/apache/directory/shared/ldap/codec/modify/ModifyRequestGrammar.java MODIFY /directory/branches/shared/0.9.5/ldap/src/main/java/org/apache/directory/shared/ldap/codec/modifyDn/ModifyDNRequestGrammar.java MODIFY /directory/branches/shared/0.9.5/ldap/src/main/java/org/apache/directory/shared/ldap/codec/bind/BindRequestGrammar.java MODIFY /directory/branches/shared/0.9.5/ldap/src/main/java/org/apache/directory/shared/ldap/codec/search/SearchRequestGrammar.java MODIFY /directory/branches/shared/0.9.5/ldap/src/main/java/org/apache/directory/shared/ldap/codec/LdapResultGrammar.java MODIFY /directory/branches/shared/0.9.5/ldap/src/main/java/org/apache/directory/shared/ldap/codec/compare/CompareRequestGrammar.java MODIFY /directory/branches/shared/0.9.5/ldap/src/main/java/org/apache/directory/shared/ldap/codec/search/SearchResultEntryGrammar.java MODIFY /directory/branches/shared/0.9.5/ldap/src/main/java/org/apache/directory/shared/ldap/codec/del/DelRequestGrammar.java

Repository	Revision	Date	User	Message
ASF	#437580	Mon Aug 28 07:14:29 UTC 2006	elecharny	Added a test to check that the correct exception is returned instead of a DecoderException (~~DIRSERVER-711~~)
				Files Changed
				MODIFY /directory/branches/shared/0.9.5/ldap/src/test/java/org/apache/directory/shared/ldap/codec/bind/BindRequestTest.java

Repository	Revision	Date	User	Message
ASF	#437586	Mon Aug 28 07:18:49 UTC 2006	elecharny	Fixed ~~DIRSERVER-711~~ for BindRequest. We now return a INVALID_DN_SYNTAX or a INVALID_CREDENTIAL instead of a PROTOCOL_ERROR in some error conditions.
				Files Changed
				MODIFY /directory/branches/shared/0.9.5/ldap/src/main/java/org/apache/directory/shared/ldap/codec/bind/BindRequestGrammar.java

Repository	Revision	Date	User	Message
ASF	#437695	Mon Aug 28 13:10:30 UTC 2006	elecharny	Fixed the throwed excpetions to fix ~~DIRSERVER-711~~
				Files Changed
				MODIFY /directory/branches/shared/0.9.5/ldap/src/main/java/org/apache/directory/shared/ldap/codec/add/AddRequestGrammar.java

Repository	Revision	Date	User	Message
ASF	#437713	Mon Aug 28 14:06:55 UTC 2006	elecharny	Fixed the throwed exceptions to fix ~~DIRSERVER-711~~
				Files Changed
				MODIFY /directory/branches/shared/0.9.5/ldap/src/main/java/org/apache/directory/shared/ldap/codec/compare/CompareRequestGrammar.java

Repository	Revision	Date	User	Message
ASF	#437727	Mon Aug 28 15:28:34 UTC 2006	elecharny	Fixed the throwed exceptions to fix ~~DIRSERVER-711~~
				Files Changed
				MODIFY /directory/branches/shared/0.9.5/ldap/src/main/java/org/apache/directory/shared/ldap/codec/modify/ModifyRequestGrammar.java

Repository	Revision	Date	User	Message
ASF	#437758	Mon Aug 28 17:21:50 UTC 2006	elecharny	Fixed the throwed exceptions to fix ~~DIRSERVER-711~~
				Files Changed
				MODIFY /directory/branches/shared/0.9.5/ldap/src/main/java/org/apache/directory/shared/ldap/codec/modifyDn/ModifyDNRequestGrammar.java

Repository	Revision	Date	User	Message
ASF	#438508	Wed Aug 30 15:00:01 UTC 2006	elecharny	- Formated the PDUs comments - Added tests for ~~DIRSERVER-711~~
				Files Changed
				MODIFY /directory/branches/shared/0.9.5/ldap/src/test/java/org/apache/directory/shared/ldap/codec/search/SearchRequestSubstringTest.java MODIFY /directory/branches/shared/0.9.5/ldap/src/test/java/org/apache/directory/shared/ldap/codec/search/SearchRequestTest.java

Repository	Revision	Date	User	Message
ASF	#438859	Thu Aug 31 10:33:43 UTC 2006	elecharny	- Merged the code with 1.0 modification done to fix ~~DIRSERVER-711~~ - Removed the RDN bad hashcode() method - Merged the modification of EMPTY_BYTES done in 1.0
				Files Changed
				MODIFY /directory/trunks/shared/ldap/src/main/java/org/apache/directory/shared/ldap/name/Rdn.java MODIFY /directory/trunks/shared/ldap/src/main/java/org/apache/directory/shared/ldap/codec/del/DelRequestGrammar.java MODIFY /directory/trunks/shared/ldap/src/test/java/org/apache/directory/shared/ldap/codec/search/SearchRequestSubstringTest.java MODIFY /directory/trunks/shared/ldap/src/test/java/org/apache/directory/shared/ldap/codec/search/SearchRequestTest.java MODIFY /directory/trunks/shared/ldap/src/test/java/org/apache/directory/shared/ldap/codec/modify/ModifyRequestTest.java MODIFY /directory/trunks/shared/ldap/src/test/java/org/apache/directory/shared/ldap/codec/add/AddRequestTest.java MODIFY /directory/trunks/shared/ldap/src/main/java/org/apache/directory/shared/ldap/message/ManageDsaITControl.java MODIFY /directory/trunks/shared/ldap/src/main/java/org/apache/directory/shared/ldap/codec/compare/CompareRequestGrammar.java MODIFY /directory/trunks/shared/ldap/src/main/java/org/apache/directory/shared/ldap/codec/LdapControlGrammar.java MODIFY /directory/trunks/shared/ldap/src/test/java/org/apache/directory/shared/ldap/codec/compare/CompareRequestTest.java MODIFY /directory/trunks/shared/ldap/src/main/java/org/apache/directory/shared/ldap/codec/add/AddRequestGrammar.java MODIFY /directory/trunks/shared/ldap/src/test/java/org/apache/directory/shared/ldap/codec/LdapMessageTest.java MODIFY /directory/trunks/shared/ldap/src/main/java/org/apache/directory/shared/ldap/codec/search/FilterGrammar.java MODIFY /directory/trunks/shared/ldap/src/main/java/org/apache/directory/shared/ldap/codec/search/SearchRequestGrammar.java MODIFY /directory/trunks/shared/ldap/src/main/java/org/apache/directory/shared/ldap/codec/modify/ModifyRequestGrammar.java MODIFY /directory/trunks/shared/ldap/src/main/java/org/apache/directory/shared/ldap/codec/ResponseCarryingException.java MODIFY /directory/trunks/shared/ldap/src/test/java/org/apache/directory/shared/ldap/codec/del/DelRequestTest.java MODIFY /directory/trunks/shared/ldap/src/main/java/org/apache/directory/shared/ldap/codec/search/SearchRequest.java MODIFY /directory/trunks/shared/ldap/src/main/java/org/apache/directory/shared/ldap/codec/Control.java MODIFY /directory/trunks/shared/ldap/src/test/java/org/apache/directory/shared/ldap/codec/modifyDn/ModifyDNRequestTest.java MODIFY /directory/trunks/shared/ldap/src/test/java/org/apache/directory/shared/ldap/codec/bind/BindRequestTest.java MODIFY /directory/trunks/shared/ldap/src/test/java/org/apache/directory/shared/ldap/codec/unbind/UnBindRequestTest.java MODIFY /directory/trunks/shared/ldap/src/test/java/org/apache/directory/shared/ldap/codec/extended/ExtendedRequestTest.java MODIFY /directory/trunks/shared/ldap/src/test/java/org/apache/directory/shared/ldap/codec/search/SearchRequestMatchingRuleAssertionTest.java MODIFY /directory/trunks/shared/ldap/src/main/java/org/apache/directory/shared/ldap/codec/abandon/AbandonRequestGrammar.java MODIFY /directory/trunks/shared/ldap/src/main/java/org/apache/directory/shared/ldap/codec/search/controls/SubEntryControlGrammar.java MODIFY /directory/trunks/shared/ldap/src/test/java/org/apache/directory/shared/ldap/codec/abandon/AbandonRequestTest.java MODIFY /directory/trunks/shared/ldap/src/main/java/org/apache/directory/shared/ldap/codec/LdapMessageGrammar.java MODIFY /directory/trunks/shared/ldap/src/main/java/org/apache/directory/shared/ldap/codec/extended/ExtendedRequestGrammar.java MODIFY /directory/trunks/shared/ldap/src/main/java/org/apache/directory/shared/ldap/codec/modifyDn/ModifyDNRequestGrammar.java MODIFY /directory/trunks/shared/ldap/src/main/java/org/apache/directory/shared/ldap/codec/bind/BindRequestGrammar.java MODIFY /directory/trunks/shared/ldap/src/main/java/org/apache/directory/shared/ldap/codec/unbind/UnBindRequestGrammar.java MODIFY /directory/trunks/shared/ldap/pom.xml MODIFY /directory/trunks/shared/ldap/src/main/java/org/apache/directory/shared/ldap/codec/search/SubstringFilterGrammar.java MODIFY /directory/trunks/shared/ldap/src/main/java/org/apache/directory/shared/ldap/codec/search/MatchingRuleAssertionGrammar.java MODIFY /directory/trunks/shared/ldap/src/main/java/org/apache/directory/shared/ldap/codec/util/LdapString.java ADD /directory/trunks/shared/ldap/src/test/java/org/apache/directory/shared/ldap/codec/LdapControlTest.java (from /directory/branches/shared/0.9.5/ldap/src/test/java/org/apache/directory/shared/ldap/codec/LdapControlTest.java)