Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 321735 - Jetty client fails with ArrayIndexOutOfBoundException if request header size is greater than the one configured
Summary: Jetty client fails with ArrayIndexOutOfBoundException if request header size ...
Status: RESOLVED FIXED
Alias: None
Product: Jetty
Classification: RT
Component: client (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 critical (vote)
Target Milestone: 7.1.x   Edit
Assignee: Greg Wilkins CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-08-04 10:55 EDT by hrishi_sir CLA
Modified: 2010-10-13 17:22 EDT (History)
3 users (show)

See Also:
gregw: iplog+

Attachments
Added a new test case to reproduce this bug (6.71 KB, application/octet-stream)
2010-08-14 11:05 EDT, hrishi_sir CLA 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description hrishi_sir CLA 2010-08-04 10:55:50 EDT 
Build Identifier: 7.1.5

The jetty-client fails with ArrayIndexOutOfBoundException , if request header size is greater than the one configured. It should ideally compare the 2 & should invoke exchange's onException even before calling the requestcomplete()

Reproducible: Always

Steps to Reproduce:
1. create a httpclient instance & setup httpexchange
2. create a request with header size > default header size ( which is 6k)
2. send a request invoking client.send(exchange)
3. this just hangs ( in the log you can see ArrayIndexOutOfBoundsException)
Comment 1 hrishi_sir CLA 2010-08-14 11:05:47 EDT 
Created attachment 176599 [details]
Added a new test case to reproduce this bug

Added a new test case to reproduce this bug
Comment 2 Greg Wilkins CLA 2010-08-16 01:53:48 EDT 
thanks for the test case.

Fixed by moving the completeHeader call within the try catch block. Also added a better exception message than array out of bounds.

checked into r2213