AdaptiveTileExtensions icon indicating copy to clipboard operation
AdaptiveTileExtensions copied to clipboard

Content in Text should be encoded

Open pekspro opened this issue 10 years ago • 0 comments

This seems to be a nice little project. I was actually planning to do something similar myself but it doesn’t look necessary :-). I hope this will be added to NuGet.

However, I’m a bit concerned a bit about this line in Text.cs:

sb.Append($">{Content}</text>");

Shouldn’t Content be encoded before it’s added to the string? Characters like < and > could cause problems if they are in Content. I might missing something because I haven’t tried the code myself. Source in TileImage may have the same problem.

pekspro avatar Jun 13 '15 12:06 pekspro