如何解决Checkmarx报告的存储XSS问题

Question

我正在处理的代码库已被Checkmarx分析，它返回了一个包含“存储的XSS”问题的报告。该问题指出：

方法GetHomepageFilterByLocaleHomepageRepo.cs从数据库中获取Select元素的数据。然后，此元素的值在没有经过适当过滤或编码的情况下流过代码，并最终在方法GetProductsByFilterType中显示给用户HomepageController.cs.这可能会启用存储跨站点脚本攻击。

是否有标准的推荐方法来解决此问题？

请参阅下面提到的两种方法的代码片段。

HomepageRepo.cs

public HomepageFilter GetHomepageFilterByLocale(int localeId)
    {
        return _context.HomepageFilter.SingleOrDefault(x => x.LocaleId == localeId);
    }

主页控制器.cs

方法是调用存储库方法的位置。

[HttpGet]
    public ActionResult GetProductsByFilterType(int locale, string filterType)
    {
        HomepageViewModel model = GetHomepageViewModel(locale, filterType);

        if (model?.Products != null)
        {
            model.Products.ForEach(p => p.Name = HttpUtility.HtmlEncode(p.Name));
            model.Products.ForEach(p => p.ImageUrl = HttpUtility.HtmlAttributeEncode(p.ImageUrl));
        }

        return Json(model, JsonRequestBehavior.AllowGet);
    }

Answer 1

您应该查看漏洞流(查看漏洞时位于屏幕右侧),并了解此漏洞涉及哪些对象。

你也可以点击小问号('？'))位于漏洞名称的右侧。它应该会告诉你如何解决它。

最后，如果仍然遇到问题，可以单击查询查看器并预览查询的确切内容。

现在：根据我自己的经验，xss漏洞可以用HttpUtility轻松修复。HtmlEncode方法。

我在想这样的事情:

HttpUtility.HtmlEncode（_context.HomepageFilter.SingleOrDefault（x=