search

mwrock / RequestReduce

Instantly makes your .net website faster by reducing the number and size of requests with almost no effort.
www.requestreduce.org
Apache License 2.0
228 stars 49 forks source link

Support application/xhtml+xml content type as well as text/html #178

Closed candrews closed 12 years ago

candrews commented 12 years ago

The Request Reduce module currently only processes responses with content type "text/html" RR would also work perfectly on XHTML - so please support the "application/xhtml+xml" content type as well.

The change needs to be made at: https://github.com/mwrock/RequestReduce/blob/master/RequestReduce/Module/RequestReduceModule.cs#L298

The new condition would be:

            if (context.Response.ContentType != "text/html" ||
                request.ContentType == "application/xhtml+xml" ||
                request.RawUrl == "/favicon.ico" ||
                IsInRRContentDirectory(context))
                return;

Thanks!

mwrock commented 12 years ago

Yeah thats trivial and makes sense. I'll get that in the next release.

From: "Craig" <reply+i-4609546-5ad3378c3b1a87046815ca22761f0932c050d620-655165@reply.githu b.com>

Sent: Wednesday, May 16, 2012 10:45 AM

To: "Matt Wrock" matt@mattwrock.com

Subject: [RequestReduce] Support application/xhtml+xml content type as well as text/html (#178)

The Request Reduce module currently only processes responses with content type "text/html" RR would also work perfectly on XHTML - so please support the "application/xhtml+xml" content type as well.

The change needs to be made at: https://github.com/mwrock/RequestReduce/blob/master/RequestReduce/Module/Req uestReduceModule.cs#L298

The new condition would be:


if (context.Response.ContentType != "text/html" ||

request.ContentType == "application/xhtml+xml" ||

request.RawUrl == "/favicon.ico" ||

IsInRRContentDirectory(context))

return;

Thanks!

Reply to this email directly or view it on GitHub:

https://github.com/mwrock/RequestReduce/issues/178