Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 338961 - [PATCH] Transfer errors when using AJP
Summary: [PATCH] Transfer errors when using AJP
Status: RESOLVED FIXED
Alias: None
Product: Jetty
Classification: RT
Component: server (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 major (vote)
Target Milestone: 7.2.x   Edit
Assignee: Greg Wilkins CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-03-04 12:31 EST by Przemek Bruski CLA
Modified: 2011-03-07 00:07 EST (History)
1 user (show)

See Also:

Attachments
patch (1.21 KB, patch)
2011-03-04 12:33 EST, Przemek Bruski CLA 		no flags Details | Diff
patch without debug (2.62 KB, patch)
2011-03-04 12:35 EST, Przemek Bruski CLA 		no flags Details | Diff
patch stripped down to include just the needed stuff (1.09 KB, patch)
2011-03-04 12:38 EST, Przemek Bruski CLA 		no flags Details | Diff
Show Obsolete (2) 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 Przemek Bruski CLA 2011-03-04 12:31:51 EST 
Build Identifier: 

When using Jetty jetty-7.2.1.v20101111 with AJP, I get the following error:
2011-03-04 17:46:53,005 ERROR [qtp22172629-13] [FiveOhOh] 500 Exception was thrown.
java.io.IOException: FULL
	at org.eclipse.jetty.ajp.Ajp13Parser.fill(Ajp13Parser.java:199)
	at org.eclipse.jetty.ajp.Ajp13Parser.parseNext(Ajp13Parser.java:567)
	at org.eclipse.jetty.ajp.Ajp13Parser$Input.blockForContent(Ajp13Parser.java:856)
	at org.eclipse.jetty.ajp.Ajp13Parser$Input.read(Ajp13Parser.java:830)

I think the reason for this problem is that the default buffer size set by Ajp13SocketConnector is set to Ajp13Packet.MAX_DATA_SIZE and Ajp13Parser needs  Ajp13Packet.MAX_PACKET_SIZE to work. The attached patch fixed the problem for me.

Reproducible: Always

Steps to Reproduce:
Use POST to send something large to the server.
Comment 1 Przemek Bruski CLA 2011-03-04 12:33:40 EST 
Created attachment 190416 [details]
patch
Comment 2 Przemek Bruski CLA 2011-03-04 12:35:44 EST 
Created attachment 190417 [details]
patch without debug
Comment 3 Przemek Bruski CLA 2011-03-04 12:38:28 EST 
Created attachment 190418 [details]
patch stripped down to include just the needed stuff
Comment 4 Greg Wilkins CLA 2011-03-06 23:33:09 EST 
I think the actual fix is to set the default buffer size correctly to the packet size and not the data size.  testing now
Comment 5 Greg Wilkins CLA 2011-03-07 00:07:36 EST 
fixed for 7.3.1