Staredit Network > Forums > Technology & Computers > Topic: Allowing Users To Use HTML
Allowing Users To Use HTML
Jan 7 2011, 8:33 am
By: A_of-s_t  

Jan 7 2011, 8:33 am A_of-s_t Post #1

aka idmontie

In the website I'm developing, I'm giving users a 500px by 500px section that they can add html to in their profile. Currently, I am stripping script, iframe, and link tags and likewise am not allowing an uneven amount of opening and closing tags (that way people don't try to had a </div> in order to break the styling.

The reason I'm allowing HTML is so that people have a lot of freedom to style this section (such as creating a background image, floating things, etc.

What other hacks might people potentially try to abuse?

One thing I've thought about is trying to position elements outside of the 500px by 500px box in order to cover up other parts of the page. Any idea how I can stop this from happening?



Personal GitHub
Starcraft GitHub Organization - Feel free to request member status!
TwitchTV

Jan 7 2011, 12:00 pm Aristocrat Post #2



<plaintext>

No but seriously, allowing HTML usage is a serious security flaw. It's better to whitelist tags than to blacklist potentially unsafe ones.



None.

Jan 7 2011, 2:40 pm rockz Post #3

ᴄʜᴇᴇsᴇ ɪᴛ!

whitelists are always and will always be inherently more secure than blacklists.



"Parliamentary inquiry, Mr. Chairman - do we have to call the Gentleman a gentleman if he's not one?"

Jan 7 2011, 3:47 pm The Starport Post #4



Be sure to whitelist attributes, too.

Edit: Obligatory.

Post has been edited 1 time(s), last time on Jan 9 2011, 6:36 am by Tuxedo-Templar.



None.

Jan 7 2011, 7:42 pm A_of-s_t Post #5

aka idmontie

Ok, sounds good. :nude:



Personal GitHub
Starcraft GitHub Organization - Feel free to request member status!
TwitchTV

Jan 12 2011, 4:52 am DavidJCobb Post #6



Strip IE-only CSS expressions from your code, too.

Code
<span style="display:expression( alert('I can rape your shit with this!') || 'inline')">O HAI</span>




None.

Jan 12 2011, 5:56 am The Starport Post #7



Sanitizing CSS opens up its own can of worms. Clickjacking, anyone?



None.

Jan 12 2011, 7:14 am DavidJCobb Post #8



Quote from name:Tuxedo-Templar
Sanitizing CSS opens up its own can of worms. Clickjacking, anyone?
Can't be overly difficult to simply parse out expression() values. Not that anyone using IE deserves a secure browsing experience, but still.



None.

Options
  Back to forum
Please log in to reply to this topic or to report it.
Members in this topic: None.
[04:56 am]
Ultraviolet -- I suppose we'll likely never know, but my guess would be that they already saw it operating successfully and there was no monetary incentive to finish the original work. And the dev cycle in old school Blizzard was so hectic, it's possible it just got forgotten about after the original game got released. Plus there's an element of existing MPQ files that were packaged with the original discs becoming outdated if they updated it. And it's not like they remade the original MPQs, they just made new ones for BW specifically
[04:26 am]
Oh_Man -- so that makes me think maybe the theory they are unfinished is not true and its a deliberate design decision, coz why not finish them wen ur making brood war?
[04:25 am]
Oh_Man -- the thing is thos buildings are from classic. that means they went ahead and made brood war without ever finishing the 'unfinished' buildings
[06:15 pm]
Ultraviolet -- Yeah he's talked about a lot of that stuff in his casts before. It seems plausible. Especially knowing how Blizzard of yesteryear operated.
[03:47 pm]
NudeRaider -- to clarify: couldn't recall the behavior for every single Protoss building but I was aware the disparity exists.
[03:43 pm]
NudeRaider -- Contained nothing new for me. Didn't know all building's behavior, but very much all unit's. Also Terran balance whine - also nothing new :lol:
[2026-6-19. : 9:57 am]
Oh_Man -- makes me wonder if SEN knows anything about the topic
[2026-6-19. : 9:57 am]
Oh_Man -- artosis dropped pretty interesting vid: https://youtu.be/EqZHB8Blra4
[2026-6-18. : 5:01 pm]
Ultraviolet -- :wob:
[2026-6-17. : 6:15 am]
RexyRex -- :wob:
Please log in to shout.


Members Online: jun3hong