I submitted a small patch recently to fix an issue where ActionController::AbstractRequest#content_type is including a trailing \r:
http://dev.rubyonrails.org/ticket/9053
However, bitsweat had written tests to check for the \r. I'm wondering if there's a reason for this, and if removing it is going to break something else.
Cheers,
-b