Doug Tangren

add integration tests for various platforms

2

unit testing has proven unreliable because it ultimately depends on ci runtime internals and how they run tests. instead it would be better to use scripted integrations tests

it would be nice to skip travis commits when just working on windows. [ci skip] also skips appveyor builds. this seems to be customizable though https://www.appveyor.com/docs/how-to/filtering-commits/

We currently define an [XForwardedFor](https://github.com/unfiltered/unfiltered/blob/master/library/src/main/scala/request/headers.scala#L175) header extractor that extracts an [X-Forwared-For](http://en.wikipedia.org/wiki/X-Forwarded-For) http header commonly used to track and collect client ips in a chain of reverse proxies. According to wikipedia...

...where it makes sense. Feedback I've gotten in the past was that it was not straight forward to find things w/o the aid of an ide or ack. It may...

Consider using underlying bindings for cookie parsing

3

We used to do this but then switched to custom parsing because we depended on am older version of the servlet spec missing some needed cookie interfaces. This is no...

we should be able to work around this [1](https://github.com/unfiltered/Unfiltered/commit/6423c54a4c697042d8e55f7d58feb324f90f9894) by using the server anylocal and maybe exposing the port info for the companion dispatch client instances

calls to `getString` [here](https://github.com/unfiltered/unfiltered/blob/0.9.0/filter-uploads/src/main/scala/request/uploads.scala#L162) result in usage of a default character set. Instead of [getString](https://commons.apache.org/proper/commons-fileupload/apidocs/org/apache/commons/fileupload/FileItem.html#getString%28%29) we should be using [getString(encoding)](https://commons.apache.org/proper/commons-fileupload/apidocs/org/apache/commons/fileupload/FileItem.html#getString%28java.lang.String%29) to support utf8 string decoding

repeatable header values have header params truncated

1

I can't remember why we did it this way but for any repeatable header we truncate params ( this key/value pairs of header values that come after `;` ) I...

I think we solved the server issue in the last release but we are still getting [reports](https://github.com/unfiltered/unfiltered/issues/224#issuecomment-52553266) of this happening in the uploads module. Not sure if it's related, but...

Our netty request bindings define a [custom query string decoder](http://blog.lunatech.com/2009/02/03/what-every-web-developer-must-know-about-url-encoding) which is very functional and cool but I just realized a few things. - As I sometimes have to remind...