Permalink
Browse files

correct Content-Disposition header to conform to RFC 2183

(date was not inside quoted string - was breaking Chrome 16 compatibility)
  • Loading branch information...
1 parent 1e581b5 commit cbef4951c412513b69a51bde086d0a7f8c463612 RDeFreitas committed Dec 14, 2011
Showing with 1 addition and 1 deletion.
  1. +1 −1 src/Web/Mvc/SerializedView.cs
@@ -140,7 +140,7 @@ public virtual void Render(ViewContext viewContext, TextWriter writer)
response.ContentEncoding = ContentEncoding;
response.AppendHeader("X-Robots-Tag", "noindex, follow, noarchive, nosnippet");
- response.AppendHeader("Content-Disposition", String.Format(DispositionType + "; filename={0}.{1}; creation-date={2:r}", action, ContentFileExtension, DateTime.UtcNow));
+ response.AppendHeader("Content-Disposition", String.Format(DispositionType + "; filename={0}.{1}; creation-date=\"{2:r}\"", action, ContentFileExtension, DateTime.UtcNow));
if (!request.IsSecureConnection)
response.Cache.SetCacheability(HttpCacheability.NoCache);

3 comments on commit cbef495

@nberardi

Can you send a pull request, so I can update the NuGet and you can get credit for the fix.

@nberardi
@rdefreitas
Owner

I sent one same day as I did the commit :)

Please sign in to comment.