chrispatterson240 / jsonp-jav Goto Github PK

Affects: v.1.0.1

It would be useful to have some unit tests for the JsonpFilter and associated 
classes. I've put some up here if you'd like to take a look:
https://gist.github.com/1282985

Please feel free to use as you see fit. The tests do introduce some new test 
scope dependencies (junit, hamcrest, mockito, spring-test etc) so you might 
prefer to trim these down.

Thanks for contributing a very useful filter.
Cheers, Ash

Filter return callback with nothing but a wrong empty string if the
following servlets choose getWriter() but not getOutputStream() for responding.

Fixed in 1.0.1

I read the source code of jsonp-java and I found In the class 
ByteArrayServletOutputStream there is a method enlarge, why shouldn't it modify 
the count field in the class. 

The methods named setBufferSize and getBufferSize in 
HttpServletResponseContentWrapper will not get the same result because the 
setBufferSize just enlarges the buf[] array in ByteArrayServletOutputStream 
instance buffer without modifing the count field in buffer and getBufferSize 
just return the count field of ByteArrayServletOutputStream instance.

I want to suppress the quoted output how to do that?

The reason why I want remove that is because Ext js is not recognizing the 
response as a valid json string. 

When the jsonp filter tries to determine whether it should execute or not, it 
calls request.getParameterMap(). This reads from the input stream. Later in my 
servlet code, I get a handle to the input stream in order to read body content. 
At this point, the stream has been read and I get no content from the input 
stream.

Are there workarounds that will allow us to reset the input stream after the 
filter reads from it?

in quote function in JsonpFilter, I can't figure out the usage of character 
variable b.